Skip to content

CI: use YAML anchors for common values#2708

Merged
aramprice merged 1 commit intomainfrom
add-yml-anchors
Apr 15, 2026
Merged

CI: use YAML anchors for common values#2708
aramprice merged 1 commit intomainfrom
add-yml-anchors

Conversation

@aramprice
Copy link
Copy Markdown
Member

@aramprice aramprice commented Apr 15, 2026

No changes to rendered pipeline.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: b8ef39f7-1cdd-4948-b33b-b68a59ba069b

📥 Commits

Reviewing files that changed from the base of the PR and between 2536597 and 0d1051b.

📒 Files selected for processing (1)
  • ci/pipeline.yml

Walkthrough

The pull request introduces YAML anchors to the CI pipeline configuration to reduce hardcoded values. Three new anchors are defined: director_ruby_version, git_user_name, and git_user_email. These anchors replace multiple instances of hardcoded parameters throughout the pipeline, including GIT_USER_NAME and GIT_USER_EMAIL values in various task configurations. The PACKAGE parameter in bump-related tasks that previously used the value ruby-3.3 is now set through the director_ruby_version anchor reference. The changes span across several jobs including bump-gems, bump-packages, and finalize-release.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete. It lacks required sections from the template including context, tests run, release notes, breaking changes, and team tags. Add the required template sections: contextual information/links, comprehensive test results, release notes description, breaking change assessment, and team tags.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: introducing YAML anchors for common values in the CI pipeline to reduce duplication.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-yml-anchors

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aramprice aramprice requested review from a team, a-hassanin and fmoehler and removed request for a team April 15, 2026 01:31
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Apr 15, 2026
@aramprice aramprice merged commit d4b6584 into main Apr 15, 2026
20 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Apr 15, 2026
@aramprice aramprice deleted the add-yml-anchors branch April 15, 2026 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant