Skip to content

ci: fix beta-release transitive-skip -- !cancelled() guard - #226

Merged
alicoding merged 1 commit into
mainfrom
fix-beta-release-skip2
Aug 18, 2026
Merged

ci: fix beta-release transitive-skip -- !cancelled() guard#226
alicoding merged 1 commit into
mainfrom
fix-beta-release-skip2

Conversation

@alicoding

Copy link
Copy Markdown
Owner

Shepherd-caught: the first genuinely-green push run skipped beta-release (0 steps) despite ci-gate succeeding — a plain if: gets the implicit success() evaluated over the TRANSITIVE needs chain, and dependency-review (PR-only) legitimately skips on pushes. !cancelled() replaces the implicit status function; the explicit ci-gate.result check stays the real gate.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd

… replaces the implicit success()

The first green push run after the job landed skipped it (0 steps)
despite a green ci-gate: a plain if gets the implicit success()
evaluated across the transitive needs chain, and dependency-review
(PR-only) legitimately skips on pushes.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd
@alicoding
alicoding enabled auto-merge (squash) August 18, 2026 04:42
@alicoding
alicoding merged commit 78c60bf into main Aug 18, 2026
26 checks passed
@alicoding
alicoding deleted the fix-beta-release-skip2 branch August 18, 2026 04:50
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