Skip to content

[CI] Cancel on concurrency#2708

Open
yaox12 wants to merge 1 commit intoNVIDIA:mainfrom
yaox12:xiny/cancel_on_concurrency
Open

[CI] Cancel on concurrency#2708
yaox12 wants to merge 1 commit intoNVIDIA:mainfrom
yaox12:xiny/cancel_on_concurrency

Conversation

@yaox12
Copy link
Member

@yaox12 yaox12 commented Feb 26, 2026

Description

Enable cancellation on concurrency to CI jobs to save CI resources, e.g., autoformatting from the pre-commit hook will trigger new CI jobs.

Type of change

  • Documentation change (change only to the documentation, either a fix or a new content)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Infra/Build change
  • Code refactoring

Signed-off-by: Xin Yao <xiny@nvidia.com>
@yaox12 yaox12 requested a review from ptrendx February 26, 2026 06:00
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 26, 2026

Greptile Summary

Added concurrency cancellation to three CI workflows (Build, Documentation, Lint) to prevent wasted CI resources when new commits are pushed to the same PR.

  • Configured concurrency groups using workflow name + PR number (or branch ref for non-PR triggers)
  • Set cancel-in-progress: true to automatically cancel outdated workflow runs
  • Implementation follows GitHub Actions best practices for CI resource optimization

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The concurrency configuration uses the standard GitHub Actions pattern correctly. Each workflow is properly scoped, and the fallback logic handles all trigger types appropriately. This is a common CI optimization with no functional impact on the codebase itself.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/build.yml Added concurrency cancellation to prevent concurrent build runs on the same PR or branch
.github/workflows/docs.yml Added concurrency cancellation to prevent concurrent documentation build runs
.github/workflows/lint.yml Added concurrency cancellation to prevent concurrent lint runs on the same PR or branch

Last reviewed commit: d17a88d

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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