Skip to content

ci: let every PR title run finish, so a cancelled run cannot block a merge - #86

Merged
NSchatz merged 1 commit into
mainfrom
ci/pr-title-no-cancel
Nov 8, 2026
Merged

NSchatz merged 1 commit into
mainfrom
ci/pr-title-no-cancel

Conversation

@NSchatz

@NSchatz NSchatz commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

pr-title / conventional-commits is a required check here. Its caller ran under a concurrency group with cancel-in-progress: true. Dependabot pushes and edits a pull request at the same moment, which starts two runs, and the group cancels one. When the cancelled run sits in the newer check suite, the required context counts as failing and the pull request stays blocked until someone re-runs it.

Setting cancel-in-progress: false would not be enough: a group still cancels a queued run when a newer one arrives. So the concurrency block is removed. Each run takes seconds, every one finishes, and the newest result stands. The shared workflow it calls has no concurrency setting of its own.

Checked locally: every command in CLAUDE.md's gate passes, run one at a time, and actionlint 1.7.12 with shellcheck 0.11.0 is clean on the changed workflow.

https://claude.ai/code/session_01MXLcW95gf97vs9RBU1YsXf

…merge

The PR title caller ran under a concurrency group with cancel-in-progress.
A push and a title edit that land together (Dependabot does both) start
two runs and one is cancelled; when the cancelled run is in the newer
check suite, the required pr-title / conventional-commits context counts
as failing until someone re-runs it. A group without cancel-in-progress
still cancels a queued run, so the block is removed: each short run
finishes and the newest result stands.

Claude-Session: https://claude.ai/code/session_01MXLcW95gf97vs9RBU1YsXf
@NSchatz
NSchatz merged commit 2874a3a into main Nov 8, 2026
13 checks passed
@NSchatz
NSchatz deleted the ci/pr-title-no-cancel branch November 8, 2026 05:00
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