Skip to content

Add concurrency groups to CI and the release workflows - #35

Merged
setoelkahfi merged 1 commit into
developmentfrom
ci/workflow-concurrency
Oct 11, 2026
Merged

setoelkahfi merged 1 commit into
developmentfrom
ci/workflow-concurrency

Conversation

@setoelkahfi

Copy link
Copy Markdown
Contributor

What

  • ci.yml: group by ref and cancel the run in flight when a newer push lands, so a burst of pushes doesn't queue a full ubuntu + macOS matrix for each one.
  • release-github.yml and release-mcp-registry.yml: group by release tag and never cancel. A second dispatch for the same tag waits for the first instead of racing it. Cancelling a half-built release or a mid-publish mcp-publisher could leave things half done.

release-crates.yml (#32) already has its own group, also without cancelling.

Checked

Only workflow files changed, so there is nothing to build locally. The CI group takes effect on the first push after this merges.

CI cancels the run in flight when a newer push lands on the same ref.
release-github and release-mcp-registry queue per tag and never cancel,
so a second dispatch for the same tag waits instead of racing the first.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@sigit-code-review sigit-code-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Added concurrency groups to CI and release workflows to control parallel runs and cancellations. The CI workflow cancels in-progress runs on the same ref to avoid queuing redundant matrix jobs. The release workflows group by release tag and do not cancel in-progress runs to prevent partial or conflicting releases. This ensures orderly and efficient workflow execution.


Automated review by siGit Code Review · commit 3684d1e · see the review dashboard

Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/release-github.yml
Comment thread .github/workflows/release-mcp-registry.yml
@setoelkahfi setoelkahfi self-assigned this Oct 11, 2026
@setoelkahfi
setoelkahfi requested a review from paydii October 11, 2026 23:29

@sigit-code-review sigit-code-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All good from my side.

@setoelkahfi
setoelkahfi merged commit 5f03e96 into development Oct 11, 2026
2 checks passed
@setoelkahfi
setoelkahfi deleted the ci/workflow-concurrency branch October 11, 2026 23: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