Skip to content

fix: use printf for PR body to avoid YAML indentation error in run block#4450

Merged
matthew-dean merged 1 commit into
masterfrom
matthew-dean-patch-3
Jun 13, 2026
Merged

fix: use printf for PR body to avoid YAML indentation error in run block#4450
matthew-dean merged 1 commit into
masterfrom
matthew-dean-patch-3

Conversation

@matthew-dean

@matthew-dean matthew-dean commented Jun 13, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

The multiline BODY= bash string spanning lines 202-204 had line 204 at column 0, which terminates YAML's block scalar early — causing a YAML parse error that prevented the workflow from running at all.

Fix: replace the multiline assignment with a single-line printf call so the content stays within the run: block's indentation level.

Summary by CodeRabbit

No User-Facing Changes

This release contains internal process improvements to release automation with no visible impact to end-users.

@matthew-dean
fix: use printf for PR body to avoid YAML indentation error in run block
913ba45 
Removed redundant line from PR body in release workflow.
@coderabbitai

coderabbitai Bot commented Jun 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 70b056e3-fc34-4982-9152-a12df26bd059

📥 Commits

Reviewing files that changed from the base of the PR and between 7a0dd6d and 913ba45.

📒 Files selected for processing (1)
  • .github/workflows/create-release-pr.yml
📝 Walkthrough

Walkthrough

The release PR workflow's BODY variable construction is changed from a multiline string with direct ${NEXT_VERSION} interpolation to a printf call with format placeholders. The resulting PR body content is functionally identical.

Changes

Release PR Workflow

Layer / File(s) Summary
BODY variable printf construction
.github/workflows/create-release-pr.yml		 BODY is now assigned via printf with %s placeholders for NEXT_VERSION instead of a multiline string with inline ${NEXT_VERSION} interpolation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 A string with a version tucked in,
Now a printf where the format begins.
Same message, same PR,
Just a cleaner little star —
Hop along, release can begin!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: replacing a multiline string assignment with printf to fix YAML indentation issues in the workflow file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 matthew-dean-patch-3

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.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 13, 2026
@matthew-dean matthew-dean merged commit d904698 into master Jun 13, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@matthew-dean