Skip to content

fix(ci): use workflow-scoped concurrency groups for PR pipelines#37

Merged
rhuanbarreto merged 1 commit into
mainfrom
fix/ci-concurrency-groups
Mar 3, 2026
Merged

fix(ci): use workflow-scoped concurrency groups for PR pipelines#37
rhuanbarreto merged 1 commit into
mainfrom
fix/ci-concurrency-groups

Conversation

@rhuanbarreto

Copy link
Copy Markdown
Contributor

Summary

  • Replace hardcoded concurrency group prefixes (code-pr-, windows-smoke-, pr-lock-) with ${{ github.workflow }}-${{ ... }} to auto-namespace by workflow name, preventing accidental group sharing if workflows are copied
  • Remove unnecessary edited event type from windows-smoke-test.yml — it doesn't validate PR titles (only code-pull-request.yml does via commitlint), so re-running the Windows smoke test on title/body edits was wasteful

Test plan

  • Open a PR and verify both Validate and Windows Smoke Test run independently without cancelling each other
  • Push a follow-up commit and verify only the previous run of each workflow is cancelled (not cross-workflow)
  • Edit the PR title and verify only Validate re-runs (not Windows Smoke Test)

🤖 Generated with Claude Code

Replace hardcoded concurrency group prefixes with ${{ github.workflow }}
to auto-namespace by workflow name, preventing accidental group sharing
if workflows are copied. Also remove unnecessary `edited` event type
from windows-smoke-test since it doesn't validate PR titles.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rhuanbarreto rhuanbarreto merged commit 0b1030c into main Mar 3, 2026
6 checks passed
@rhuanbarreto rhuanbarreto deleted the fix/ci-concurrency-groups branch March 3, 2026 19:52
@archgatebot archgatebot Bot mentioned this pull request Mar 3, 2026
rhuanbarreto added a commit that referenced this pull request Mar 3, 2026
Replace hardcoded concurrency group prefixes with ${{ github.workflow }}
to auto-namespace by workflow name, preventing accidental group sharing
if workflows are copied. Also remove unnecessary `edited` event type
from windows-smoke-test since it doesn't validate PR titles.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
rhuanbarreto added a commit that referenced this pull request Mar 4, 2026
Replace hardcoded concurrency group prefixes with ${{ github.workflow }}
to auto-namespace by workflow name, preventing accidental group sharing
if workflows are copied. Also remove unnecessary `edited` event type
from windows-smoke-test since it doesn't validate PR titles.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Mar 4, 2026
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