Skip to content

Enable concurrency for each workflow#958

Merged
5ouma merged 1 commit into
mainfrom
ci-actions-concurrency
May 16, 2026
Merged

Enable concurrency for each workflow#958
5ouma merged 1 commit into
mainfrom
ci-actions-concurrency

Conversation

@5ouma
Copy link
Copy Markdown
Owner

@5ouma 5ouma commented May 16, 2026

Cancel in-progress jobs when a new commit is pushed to the same PR.

Copilot AI review requested due to automatic review settings May 16, 2026 08:25
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a concurrency block to each GitHub Actions workflow so that newer runs cancel in-progress runs from the same PR/ref, saving CI minutes and providing faster feedback.

Changes:

  • Adds concurrency groups keyed by PR number (falling back to github.ref) for PR-driven workflows (ci, pre-commit, labeler, label-past-pr).
  • Adds a github.workflow+github.ref concurrency group for the release workflow.
  • Adds a github.event.schedule-based concurrency group for the scheduled gist-update workflow.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/ci.yml Per-PR concurrency with cancel-in-progress.
.github/workflows/pre-commit.yml Per-PR concurrency with cancel-in-progress.
.github/workflows/labeler.yml Per-PR concurrency with cancel-in-progress.
.github/workflows/label-past-pr.yml Per-PR concurrency with cancel-in-progress.
.github/workflows/release.yml Per-ref concurrency; cancelling release mid-run may be unsafe.
.github/workflows/gist-update.yml Per-schedule concurrency with cancel-in-progress.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/gist-update.yml
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

Review Change Stack

Warning

Rate limit exceeded

@5ouma has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 54 minutes and 17 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 95454234-90fd-4630-a781-437b3b4d2ffe

📥 Commits

Reviewing files that changed from the base of the PR and between 3c983fc and ffd9e4a.

📒 Files selected for processing (5)
  • .github/workflows/ci.yml
  • .github/workflows/gist-update.yml
  • .github/workflows/label-past-pr.yml
  • .github/workflows/labeler.yml
  • .github/workflows/pre-commit.yml

Walkthrough

This pull request adds concurrency configuration blocks to six GitHub workflows. Each block groups runs by workflow name and event context (pull request number or git reference) and enables cancel-in-progress to automatically cancel older in-progress runs when newer runs start for the same group.

Changes

Workflow Concurrency Control

Layer / File(s) Summary
Concurrency control across workflows
.github/workflows/ci.yml, .github/workflows/gist-update.yml, .github/workflows/label-past-pr.yml, .github/workflows/labeler.yml, .github/workflows/pre-commit.yml, .github/workflows/release.yml
All six workflows now define concurrency policies scoped by workflow name and either the pull request number (with fallback to git reference) or git reference alone. Each enables automatic cancellation of in-progress runs to prevent duplicate execution for the same logical group.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Enable concurrency for each workflow' directly and clearly describes the main change across all workflow files.
Description check ✅ Passed The description 'Cancel in-progress jobs when a new commit is pushed to the same PR' accurately describes the purpose and effect of the concurrency configuration added.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci-actions-concurrency

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 and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

CodeRabbit couldn't request changes on this pull request because it doesn't have sufficient GitHub permissions.

Please grant CodeRabbit Pull requests: Read and write permission and re-run the review.

👉 Steps to fix this

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/release.yml:
- Around line 11-14: The release workflow's concurrency block currently sets
cancel-in-progress: true which can abort non-idempotent release steps (e.g.,
Songmu/tagpr runs) and leave tags/PRs in a partial state; update the concurrency
configuration used by the release workflow so that cancel-in-progress is
disabled (set cancel-in-progress: false or remove the cancel-in-progress key)
for the concurrency group defined by group: ${{ github.workflow }}-${{
github.ref }}, ensuring release runs are serialized and not auto-cancelled.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 68c4967f-cfb4-49ba-b51d-dc4c524f45fc

📥 Commits

Reviewing files that changed from the base of the PR and between a36fe4a and 3c983fc.

📒 Files selected for processing (6)
  • .github/workflows/ci.yml
  • .github/workflows/gist-update.yml
  • .github/workflows/label-past-pr.yml
  • .github/workflows/labeler.yml
  • .github/workflows/pre-commit.yml
  • .github/workflows/release.yml

Comment thread .github/workflows/release.yml Outdated
Cancel in-progress jobs when a new commit is pushed to the same PR.
@5ouma 5ouma force-pushed the ci-actions-concurrency branch from 3c983fc to ffd9e4a Compare May 16, 2026 08:30
@5ouma 5ouma merged commit 82c372a into main May 16, 2026
9 checks passed
@5ouma 5ouma deleted the ci-actions-concurrency branch May 16, 2026 08:32
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.

2 participants