Skip to content

fix(ci): cancel superseded pull request validation - #22

Merged
dnikolayev merged 1 commit into
mainfrom
fix/pr-workflow-supersession-20261011-mdi61pom
Oct 10, 2026
Merged

dnikolayev merged 1 commit into
mainfrom
fix/pr-workflow-supersession-20261011-mdi61pom

Conversation

@dnikolayev

@dnikolayev dnikolayev commented Oct 10, 2026 •

Copy link
Copy Markdown
Owner

Cancel superseded validation runs for the same pull request. The concurrency group includes the workflow, event, and PR number; other events use a unique run ID.

Only PR and PR-target events enable cancellation. Existing jobs, permissions, cleanup, and release behavior are preserved.

Validation: focused YAML parsing, unchanged-workflow parity checks, and PR/non-PR concurrency identity checks passed. Full validation runs in GitHub Actions.

Summary by CodeRabbit

  • Chores
    • CI runs for pull requests are grouped and in-progress runs are canceled when a newer run starts. Other event types are unaffected.

@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: d0f52d13-768f-4265-a2c8-d91378ce14bf

📥 Commits

Reviewing files that changed from the base of the PR and between 135f325 and 879ab33.


📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

The CI workflow now groups runs by workflow, event type, and pull-request number, using the run ID when no pull-request number is available. It cancels in-progress runs for pull_request and pull_request_target events.

Changes

CI workflow concurrency

Layer / File(s) Summary
Configure workflow concurrency
.github/workflows/ci.yml
Runs share a concurrency group when their workflow, event type, and pull-request number match. Runs without a pull-request number use the run ID. In-progress runs are canceled for pull_request and pull_request_target events.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix


Merge Risk: ⚪ Minimal · up to 879ab

Superseded pull request validation runs can be canceled without canceling main-branch push runs. No merge-blocking issue remains.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: canceling superseded pull request validation runs in CI.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dnikolayev
dnikolayev merged commit 6fb0477 into main Oct 10, 2026
5 checks passed
@dnikolayev
dnikolayev deleted the fix/pr-workflow-supersession-20261011-mdi61pom branch October 10, 2026 22:46
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