Skip to content

ci: add concurrency group to cancel superseded workflow runs (#229) - #338

Merged
blockchain-maxis merged 3 commits into
blockchain-maxis:mainfrom
Lazyartist1:fix/ci-concurrency-group-229
Sep 2, 2026
Merged

ci: add concurrency group to cancel superseded workflow runs (#229)#338
blockchain-maxis merged 3 commits into
blockchain-maxis:mainfrom
Lazyartist1:fix/ci-concurrency-group-229

Conversation

@Lazyartist1

Copy link
Copy Markdown
Contributor

Closes #229

Summary of Changes

Adds a concurrency group to .github/workflows/ci.yml keyed on the workflow name and target ref (github.head_ref || github.ref) with cancel-in-progress: true.

Context & Benefits

  • Prevents redundant, queued, or superseded workflow runs on rapid successive pushes to open PR branches or tracking branches.
  • Cancels in-progress runs as soon as a new commit is pushed to the same branch/ref, preserving CI resources and avoiding wasted run time.

Verification Status

  • Docs Check: node scripts/check-docs.mjs passed (22 markdown files, 27 env vars, no drift).
  • Contract Error Check: node scripts/check-contract-errors.mjs passed (9 variants matched).
  • Test Suite: Node test runner passes across apps/web/lib/*.test.ts.

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Lazyartist1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@netlify

netlify Bot commented Aug 30, 2026

Copy link
Copy Markdown

Deploy Preview for stellar-signet ready!

Name Link
🔨 Latest commit da36bf9
🔍 Latest deploy log https://app.netlify.com/projects/stellar-signet/deploys/6a9802c44b38a10008236d62
😎 Deploy Preview https://deploy-preview-338--stellar-signet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Ipramking is attempting to deploy a commit to the blockchainmaxis-8449's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Lazyartist1
Lazyartist1 force-pushed the fix/ci-concurrency-group-229 branch from 6e1a340 to f309bd2 Compare August 30, 2026 16:10
The concurrency group is right, but cancel-in-progress: true applies to the
push trigger too. A run on main is the record of whether that merged commit is
green, and merges landing close together would cancel the earlier commit's run
before it reported — which is exactly what happens while working through a
queue of PRs. Scope the cancellation to pull_request.
@blockchain-maxis
blockchain-maxis merged commit 64385d5 into blockchain-maxis:main Sep 2, 2026
10 of 11 checks passed
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.

CI has no concurrency group, so superseded runs are never cancelled

2 participants