Skip to content

fix(ci): fix YAML syntax error in sync-manifest action breaking release-please#97

Merged
MoonBoi9001 merged 1 commit intomainfrom
fix/sync-manifest-yaml-syntax
Jan 16, 2026
Merged

fix(ci): fix YAML syntax error in sync-manifest action breaking release-please#97
MoonBoi9001 merged 1 commit intomainfrom
fix/sync-manifest-yaml-syntax

Conversation

@MoonBoi9001
Copy link
Member

@MoonBoi9001 MoonBoi9001 commented Jan 16, 2026

Summary

Fixes a YAML syntax error in the sync-manifest action introduced in #96 that has been breaking the release-please workflow.

The markdown list items (- **Previous version:**) in the gh pr create --body argument were being interpreted as YAML list items by the GitHub Actions parser, causing the workflow to fail with:

yaml: line 104: could not find expected ':'

Note: release-please has actually been failing since #74 introduced sync-manifest - initially due to branch protection blocking direct pushes to main, then after #96's fix attempt, due to this YAML syntax error.

Changes

  • Use printf to construct the PR body as a variable, keeping all content on properly indented lines
  • This prevents the YAML parser from misinterpreting markdown list items as YAML structure

Generated with Claude Code

The markdown list items in the gh pr create --body argument were being
interpreted as YAML list items by the GitHub Actions parser, causing
the sync-manifest action to fail with YAML parsing errors.

Using printf to construct the PR body keeps all content on properly
indented lines, preventing YAML parser confusion.

Validated with actionlint and Python yaml parser.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@MoonBoi9001 MoonBoi9001 force-pushed the fix/sync-manifest-yaml-syntax branch from 7462f55 to 545d18f Compare January 16, 2026 19:27
@MoonBoi9001 MoonBoi9001 marked this pull request as ready for review January 16, 2026 19:35
@MoonBoi9001 MoonBoi9001 changed the title fix(ci): use heredoc for PR body to avoid YAML parsing error fix(ci): fix YAML syntax error in sync-manifest action breaking release-please Jan 16, 2026
@MoonBoi9001
Copy link
Member Author

This should finally fix release please 🤞

@MoonBoi9001 MoonBoi9001 merged commit cd07a6d into main Jan 16, 2026
12 checks passed
@MoonBoi9001 MoonBoi9001 deleted the fix/sync-manifest-yaml-syntax branch January 16, 2026 21:48
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.

2 participants