Skip to content

fix(ci): repair staging-ci workflow parsing#1090

Merged
henrypark133 merged 1 commit intostagingfrom
fix/staging-ci-workflow-parsing
Mar 12, 2026
Merged

fix(ci): repair staging-ci workflow parsing#1090
henrypark133 merged 1 commit intostagingfrom
fix/staging-ci-workflow-parsing

Conversation

@henrypark133
Copy link
Copy Markdown
Collaborator

This repairs the staging-ci.yml parse failure introduced in c7dec64.

The workflow was failing before any jobs were created because a multiline shell assignment in the promotion PR commit-summary block broke YAML parsing.

Validation:

  • actionlint -ignore 'SC2001|SC2129' .github/workflows/staging-ci.yml .github/workflows/test.yml .github/workflows/e2e.yml

Copilot AI review requested due to automatic review settings March 12, 2026 23:55
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added scope: ci CI/CD workflows size: XS < 10 changed lines (excluding docs) risk: medium Business logic, config, or moderate-risk modules contributor: core 20+ merged PRs labels Mar 12, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a YAML parsing failure in the staging CI workflow by replacing a multiline shell variable assignment (which broke YAML parsing) with a single-line Bash-safe concatenation.

Changes:

  • Replace a multiline COMMIT_MD assignment with COMMIT_MD+=... using $'\n' to append a newline without introducing invalid YAML.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: core 20+ merged PRs risk: medium Business logic, config, or moderate-risk modules scope: ci CI/CD workflows size: XS < 10 changed lines (excluding docs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants