Repository navigation
Add cancel-superseded caller - #29
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019xfc9ojH5gMAJaAFRMPitW
|
Claude finished review — View run I fetched PR 29 live just now. It is open, not a draft, not merged, and its head is FindingsNone. The diff adds one workflow file, VerdictReady for merge Structured Review Data (JSON){
"schema_version": "1.1",
"reviewer": "claude",
"commit_sha": "c26092e8a60827545e2c63822711f0a484a33f34",
"verdict": "CLEAN",
"findings": [],
"detailed_assessment": "The new cancel-superseded.yml caller matches the gha example exactly: same triggers, the actions write and pull-requests read grants, and the @v3 pin used by sibling callers.",
"holistic_assessment": "The change is scoped to a single new workflow with no content, math, or build-artifact changes, and it does not touch the publish or preview pipelines."
}Reviewed commit: c26092e |
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues were identified for the workflow change to cancel stale pull-request CI runs.
No open issues 🎉
What changed in this PR
Adds a caller to the shared Morrison-Lab workflow to cancel stale PR CI runs, while leaving runs not associated with a PR unaffected.
Changes:
- Triggers cancellation when a pull request is synchronized or closed.
- Calls
Morrison-Lab/gha’s reusable workflow atv3with Actions write and pull-request read permissions.
| File | Description |
|---|---|
.github/workflows/cancel-superseded.yml |
Adds the event triggers, permissions, and shared cancellation workflow call. |
Review effort: Balanced
2026-11-05 07:20 UTC |
Ezra's standing rule (2026-11-02): "always cancel stale runs". On Ezra's decision (2026-11-03) to roll this out to every Morrison-Lab repo with PR CI, this adds a 17-line caller for
Morrison-Lab/gha'scancel-superseded.yml(gha#1400, gha#1401, onv3). On everysynchronizeandclosedof a pull request it cancels that PR's queued and running runs for any head other than the current one, and every earlier run of a closed PR. Runs not tied to a PR (push tomain) are not covered. Not exercised before merge: the first live run is this PR's own next push or close.Generated by Claude Code