Skip to content

fix: release notes block was invalid YAML + workflow-lint gate - #88

Merged
alicoding merged 2 commits into
mainfrom
fix/release-notes-yaml
Aug 13, 2026
Merged

fix: release notes block was invalid YAML + workflow-lint gate#88
alicoding merged 2 commits into
mainfrom
fix/release-notes-yaml

Conversation

@alicoding

Copy link
Copy Markdown
Owner

v0.2.0's first release run failed with zero jobs executed — GitHub couldn't parse release.yml: PR #86's multi-line --notes string had lines at column 0 inside a run: | block, which terminates the literal scalar. GitHub only parses a workflow when its trigger fires, so tag-only workflows can merge broken through green CI — hence the new gate: check-workflow-yaml.sh (parse-only, python3, lefthook + CI same script) + actionlint in a new workflow-lint job wired into the required CI gate. After merge: re-point v0.2.0 and re-fire.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ

The v0.2.0 run failed with zero jobs: --notes' multi-line string put
continuation lines at column 0 inside a run:| literal block, which
terminates the scalar and breaks the document. Notes now build
line-by-line (indented) into a notes-file. Gate added so the class is
caught at commit time, not tag time: scripts/check-workflow-yaml.sh
(parse every workflow; lefthook + CI, one script both callers) plus
actionlint in CI's new workflow-lint job, wired into ci-gate.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ
@alicoding
alicoding enabled auto-merge (squash) August 13, 2026 21:19
…PL-3

dependency-review deny-licenses flagged arduino/setup-task (the action
is GPL-3.0; Task itself is MIT) once this PR touched release.yml's
uses: lines. go install of the MIT tool, version-pinned, drops both
the license hit and a third-party action.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ
@alicoding
alicoding merged commit 39fc071 into main Aug 13, 2026
24 checks passed
@alicoding
alicoding deleted the fix/release-notes-yaml branch August 14, 2026 00:11
alicoding added a commit that referenced this pull request Aug 15, 2026
* fix: release notes block was invalid YAML + workflow-lint gate

The v0.2.0 run failed with zero jobs: --notes' multi-line string put
continuation lines at column 0 inside a run:| literal block, which
terminates the scalar and breaks the document. Notes now build
line-by-line (indented) into a notes-file. Gate added so the class is
caught at commit time, not tag time: scripts/check-workflow-yaml.sh
(parse every workflow; lefthook + CI, one script both callers) plus
actionlint in CI's new workflow-lint job, wired into ci-gate.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ

* fix: install Task via go install — the setup-task action wrapper is GPL-3

dependency-review deny-licenses flagged arduino/setup-task (the action
is GPL-3.0; Task itself is MIT) once this PR touched release.yml's
uses: lines. go install of the MIT tool, version-pinned, drops both
the license hit and a third-party action.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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