CI: fix YAML syntax error in pr-test-builds workflow#11384
Merged
sensei-hacker merged 1 commit intoiNavFlight:maintenance-9.xfrom Mar 1, 2026
Merged
Conversation
Multi-line shell string in the NOTES variable had unindented continuation
lines (**..., >...) that were less indented than the YAML block scalar
body, causing a parse error on every run.
Replace with printf + --notes-file to avoid multi-line strings in the
YAML block scalar entirely. Also moves ${{ }} expressions into env vars
rather than inline in the shell script.
Contributor
Review Summary by QodoFix YAML syntax error in pr-test-builds workflow
WalkthroughsDescription• Fixes YAML syntax error in pr-test-builds workflow caused by unindented continuation lines • Replaces multi-line NOTES variable with printf writing to temp file • Moves GitHub expression variables into env section for better maintainability • Uses --notes-file instead of inline --notes parameter Diagramflowchart LR
A["Multi-line NOTES variable<br/>with unindented lines"] -->|Replace with| B["printf to<br/>release-notes.md"]
C["Inline ${{ }} expressions"] -->|Move to| D["env section variables"]
B -->|Use| E["--notes-file parameter"]
D -->|Reference in| F["Shell script"]
File Changes1. .github/workflows/pr-test-builds.yml
|
Contributor
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewⓘ The new review experience is currently in Beta. Learn more |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes a YAML parse error introduced in #11381 that caused the
pr-test-buildsworkflow to fail on every run with "workflow file issue".Root cause
The
NOTESvariable assignment in the "Create PR release" step spanned multiple lines, and the continuation lines (**...,>...) had zero indentation. YAML block scalars require all non-blank content lines to be at least as indented as the block's base level. Lines at column 0 inside a 10-space-indented block scalar are a parse error.Fix
Replace the multi-line
NOTESvariable +--noteswithprintfwriting to a temp file +--notes-file. This avoids multi-line strings in the YAML block scalar entirely. The${{ }}expressions are also moved intoenv:vars rather than inline in the shell script.Testing
PR #11383 had all CI checks pass before this fix was available — it can be used to verify the workflow runs correctly once this is merged.