Skip to content

Commit 0b1030c

Browse files
rhuanbarretoclaude
andauthored
fix(ci): use workflow-scoped concurrency groups for PR pipelines (#37)
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>
1 parent 3b0c3c3 commit 0b1030c

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/code-pull-request.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ on:
77
- main
88

99
concurrency:
10-
group: code-pr-${{ github.event.pull_request.number }}
10+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
1111
cancel-in-progress: true
1212

1313
jobs:

.github/workflows/update-lock.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ on:
77
- "package.json"
88
- ".github/workflows/update-lock.yaml"
99
concurrency:
10-
group: pr-lock-${{ github.event.pull_request.number }}
10+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
1111
cancel-in-progress: true
1212
jobs:
1313
update-lock:

.github/workflows/windows-smoke-test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@ name: Windows Smoke Test
22

33
on:
44
pull_request:
5-
types: [opened, edited, synchronize, reopened]
5+
types: [opened, synchronize, reopened]
66
branches:
77
- main
88

99
concurrency:
10-
group: windows-smoke-${{ github.event.pull_request.number }}
10+
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
1111
cancel-in-progress: true
1212

1313
jobs:

0 commit comments

Comments
 (0)