Skip to content

fix(release): publish after skipped rebuild - #80

Merged
rabii-chaarani merged 1 commit into
mainfrom
codex/fix-release-after-ci
Aug 17, 2026
Merged

fix(release): publish after skipped rebuild#80
rabii-chaarani merged 1 commit into
mainfrom
codex/fix-release-after-ci

Conversation

@rabii-chaarani

Copy link
Copy Markdown
Owner

Summary

  • keep Release triggered only by completed CI runs on main
  • prevent the intentionally skipped rebuild job from suppressing asset and crate publication
  • require successful direct prerequisites and affirmative publication authorization
  • add workflow policy regressions for both publisher jobs

Verification

  • cargo run -p xtask --locked -- check-workflows
  • cargo run -p xtask --locked -- release-gate
  • cargo test -p xtask --locked
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features --locked -- -D warnings

Incident

In Release run 31993773993 attempt 1, CI had already completed successfully and artifact validation passed. The rebuild job was correctly skipped because retained CI artifacts were promoted, but GitHub propagated that skipped ancestor and silently skipped both publication jobs. This change explicitly overrides only that inherited skipped status while retaining fail-closed result checks.

@rabii-chaarani
rabii-chaarani marked this pull request as ready for review August 17, 2026 04:54
@rabii-chaarani
rabii-chaarani merged commit 392b26c into main Aug 17, 2026
9 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