Skip to content

Commit daf1b21

Browse files
alicodingclaude
andcommitted
ci: beta-release must not inherit the transitive-skip -- !cancelled() 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
1 parent 17d0618 commit daf1b21

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -482,7 +482,13 @@ jobs:
482482
# release.yml's build-macos/release jobs already carry.
483483
beta-release:
484484
needs: ci-gate
485-
if: github.event_name == 'push' && github.ref == 'refs/heads/main' && needs.ci-gate.result == 'success'
485+
# !cancelled() replaces the implicit success() status function --
486+
# without it, any legitimately-skipped job in the TRANSITIVE needs
487+
# chain (dependency-review is PR-only, changes-filtered jobs skip
488+
# on docs-only pushes) auto-skips this job even when ci-gate itself
489+
# succeeded; observed on the first green push run after the job
490+
# landed (skipped, 0 steps, green gate).
491+
if: ${{ !cancelled() && github.event_name == 'push' && github.ref == 'refs/heads/main' && needs.ci-gate.result == 'success' }}
486492
runs-on: macos-latest
487493
timeout-minutes: 30
488494
permissions:

0 commit comments

Comments
 (0)