Skip to content

git Fix placement of YAML anchors to appease YTT#559

Merged
aramprice merged 1 commit into
ubuntu-jammyfrom
ugh-ytt
Apr 14, 2026
Merged

git Fix placement of YAML anchors to appease YTT#559
aramprice merged 1 commit into
ubuntu-jammyfrom
ugh-ytt

Conversation

@aramprice

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

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: 6457dbb1-8fd2-457b-b6db-6702dc9e5afd

📥 Commits

Reviewing files that changed from the base of the PR and between 220a0d6 and 44cc867.

📒 Files selected for processing (2)
  • ci/pipelines/builder.yml
  • ci/pipelines/publisher.yml

Walkthrough

The changes reorganize YAML directive and anchor declarations across two pipeline configuration files. In ci/pipelines/builder.yml, ytt directive lines were moved to appear after the anchors block with added blank line separation. In ci/pipelines/publisher.yml, a top-level anchors block defining ci_bot references was removed from the file beginning and reintroduced later under a document boundary separator, with all existing references to those anchors preserved. No job definitions, templates, or resource logic were altered.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a brief description explaining why the YAML anchor placement needed to be changed and how it resolves YTT compatibility issues.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'git Fix placement of YAML anchors to appease YTT' clearly describes the main change: reordering YAML anchor directives to comply with YTT requirements.
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 ugh-ytt

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 merged commit 6f0784f into ubuntu-jammy Apr 14, 2026
12 checks passed
@aramprice aramprice deleted the ugh-ytt branch April 14, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant