Skip to content

ci: fix main.yml YAML (quote step names containing ': ') - #330

Merged
ZhangHanDong merged 2 commits into
mainfrom
fix/ci-workflow-yaml
Aug 19, 2026
Merged

ci: fix main.yml YAML (quote step names containing ': ')#330
ZhangHanDong merged 2 commits into
mainfrom
fix/ci-workflow-yaml

Conversation

@ZhangHanDong

Copy link
Copy Markdown

All .github/workflows/main.yml runs on main since #326 fail with "This run likely failed because of a workflow file issue": two step names contained an unquoted : (Tests (workspace: lib, …), Static UX audit (gate policy: …)), which YAML parses as a nested mapping (mapping values are not allowed here, line 137). Because GitHub could not parse the file, PRs simply had no spec gate check — the breakage was invisible rather than red. The Robrix Build CI … cancelled entries are just concurrency cancellations.

  • Quote both names (file now parses; jobs: clippy, typos, check_patches, spec_gate).
  • tests/ci_policy.rs: new ci_workflow_scalars_with_colon_space_are_quoted — any single-line name:/run:/if:/key: scalar containing : must be quoted; mutation-checked (unquoting one name fails the test).

🤖 Generated with Claude Code

ZhangHanDong and others added 2 commits August 19, 2026 20:04
…ainst it

GitHub refused to parse .github/workflows/main.yml since #326 — two step
names ("Tests (workspace: lib, ...)", "Static UX audit (gate policy: ...)")
contained an unquoted ': ', which YAML reads as a nested mapping. Every push
to main since then failed with "workflow file issue" and PRs showed no
`spec gate` check at all (the file could not even be scheduled), so the
breakage looked green.

Quote both names and add `ci_workflow_scalars_with_colon_space_are_quoted`
to tests/ci_policy.rs so an unquoted ': ' scalar in name/run/if/key lines
fails `cargo test` locally before it reaches CI.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…rd was inert)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ZhangHanDong
ZhangHanDong merged commit 2f49397 into main Aug 19, 2026
13 checks passed
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