Skip to content

ci: use single quotes in the ci-green join expression - #6

Merged
TheHefty merged 1 commit into
mainfrom
fix/ci-green-expression
Aug 3, 2026
Merged

ci: use single quotes in the ci-green join expression#6
TheHefty merged 1 commit into
mainfrom
fix/ci-green-expression

Conversation

@TheHefty

@TheHefty TheHefty commented Aug 3, 2026

Copy link
Copy Markdown
Owner

fd3c7e4 wrote join(needs.*.result, " "), which is valid YAML but invalid as a GitHub Actions expression — the expression language accepts single quotes only. The whole workflow failed to parse, so the run on main showed up named .github/workflows/ci.yml instead of CI and no job ran.

Caught by actionlint:

ci.yml:93:48: got unexpected character '"' while lexing expression ...
only single quotes are available for string delimiter [expression]

A YAML parser cannot catch this class of error — the expression language sits a layer above YAML.

This is also the first PR through the new branch protection on main, so ci-green reporting green here is what proves the aggregate job actually works.

🤖 Generated with Claude Code

fd3c7e4 wrote join(needs.*.result, " "), which is valid YAML but invalid as a
GitHub Actions expression: the expression language accepts single quotes only.
The workflow failed to parse, so the run showed up named after the file rather
than "CI" and no job ran at all.

Caught by actionlint:

  ci.yml:93:48: got unexpected character '"' while lexing expression ...
  only single quotes are available for string delimiter [expression]

Worth knowing for anything added here later: a YAML parser cannot catch this
class of error, since the expression language sits a layer above YAML.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@TheHefty
TheHefty merged commit b5ffd1c into main Aug 3, 2026
15 checks passed
@TheHefty
TheHefty deleted the fix/ci-green-expression branch August 3, 2026 09:25
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