Skip to content

Commit d66a37f

Browse files
HetCreepclaude
andcommitted
fix(ci): re-sync the five workflows with TheColliery/.github templates/published-code (UMB-123 A)
Source of truth: TheColliery/.github 10f97a7. Per file: - ci / codeql / markdownlint / scorecard: gain the UMB-053 `concurrency:` groups this repo was seeded before (source -> repo). cancel-in-progress true for ci, codeql, markdownlint; false for scorecard (it publishes to the OpenSSF API). - dependabot-auto-merge: gains its per-PR concurrency group with cancel-in-progress true (main's r29 ruling; the source was corrected to match in the same unit). - The codeql-action pins (b96794f, v4.38.0, Dependabot's own PR #1) were already here and are unchanged -- the source took them from this repo, not the reverse. After this, only NOTICE (the source's {{TOKEN}} slots filled) and README.md (the generated banner) differ from the source, both by design. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent ef94b6b commit d66a37f

5 files changed

Lines changed: 36 additions & 0 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,12 @@ on:
2525
permissions:
2626
contents: read
2727

28+
# cancel-in-progress: TRUE -- ci.yml only reads, tests and compares; it publishes nothing and
29+
# touches no external state, so a superseded run has nothing to leave half-done.
30+
concurrency:
31+
group: ci-${{ github.ref }}
32+
cancel-in-progress: true
33+
2834
jobs:
2935
gate:
3036
name: gate (${{ matrix.os }} · node ${{ matrix.node }})

‎.github/workflows/codeql.yml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,13 @@ on:
1919
permissions:
2020
contents: read
2121

22+
# cancel-in-progress: TRUE -- a superseded CodeQL run simply never uploads its SARIF, and the
23+
# previous analysis stands; nothing is left half-written. The newer push analyzes the newer tree,
24+
# which is the state anyone cares about.
25+
concurrency:
26+
group: codeql-${{ github.ref }}
27+
cancel-in-progress: true
28+
2229
jobs:
2330
analyze:
2431
name: analyze (javascript)

‎.github/workflows/dependabot-auto-merge.yml‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,15 @@ on: pull_request
1414
permissions:
1515
contents: read
1616

17+
# cancel-in-progress: TRUE -- the test is ATOMICITY, not read-vs-write. Enabling auto-merge
18+
# (`gh pr merge --auto`) is IDEMPOTENT within one PR's own group, so a cancelled run corrupts
19+
# nothing: the next push to that same PR simply re-enables it. Per-PR grouping, never per-ref:
20+
# two different Dependabot PRs must not serialise behind each other (the ruling lives in
21+
# TheColliery/.github SKILL-REPO-PATTERN.md's `concurrency:` bullet).
22+
concurrency:
23+
group: dependabot-auto-merge-${{ github.event.pull_request.number }}
24+
cancel-in-progress: true
25+
1726
jobs:
1827
auto-merge:
1928
if: ${{ github.event.pull_request.user.login == 'dependabot[bot]' }}

‎.github/workflows/markdownlint.yml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,12 @@ on:
1313
permissions:
1414
contents: read
1515

16+
# cancel-in-progress: TRUE -- a lint run reads files and publishes nothing, so a superseded
17+
# run has no partial state to leave behind.
18+
concurrency:
19+
group: markdownlint-${{ github.ref }}
20+
cancel-in-progress: true
21+
1622
jobs:
1723
lint:
1824
runs-on: ubuntu-latest

‎.github/workflows/scorecard.yml‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,14 @@ on:
1313
permissions:
1414
contents: read
1515

16+
# cancel-in-progress: FALSE, deliberately. Unlike the read-only workflows here, Scorecard
17+
# UPLOADS a SARIF to code scanning and (with publish_results) posts to the OpenSSF API --
18+
# outward writes, where a cancellation mid-publish is a partially-applied result rather than a
19+
# discarded one. It is also schedule-driven, so overlap is rare and queueing costs nothing.
20+
concurrency:
21+
group: scorecard-${{ github.ref }}
22+
cancel-in-progress: false
23+
1624
jobs:
1725
analysis:
1826
name: Scorecard analysis

0 commit comments

Comments
 (0)