Skip to content

Commit 8a88a21

Browse files
patocallaghanclaude
andcommitted
Trim verbose comments on the hardening changes
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent bcc1707 commit 8a88a21

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/publish.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ permissions:
88
contents: read # workflow default (least privilege); only stage-publish also needs id-token, granted on that job
99

1010
concurrency:
11-
group: publish-${{ github.workflow }} # serialize all publish runs; never two staged releases racing for a dist-tag
12-
cancel-in-progress: false # queue, don't cancel: killing a half-done `npm stage publish` is the torn state we're avoiding
11+
group: publish-${{ github.workflow }} # serialize publishes; no dist-tag races
12+
cancel-in-progress: false # queue, don't kill an in-flight publish
1313

1414
jobs:
1515
verify:
@@ -18,7 +18,7 @@ jobs:
1818
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
1919
with:
2020
persist-credentials: false
21-
fetch-depth: 0 # full history so origin/<default> ancestry is computable; checkout fetches authenticated before stripping creds
21+
fetch-depth: 0 # full history for the ancestry check below
2222
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
2323
with:
2424
node-version-file: '.nvmrc' # pin >= 22.14.0
@@ -40,7 +40,7 @@ jobs:
4040
stage-publish:
4141
needs: verify
4242
runs-on: ubuntu-latest
43-
timeout-minutes: 15 # bound a hung publish instead of running to the 6h default
43+
timeout-minutes: 15 # cap a hung publish
4444
permissions:
4545
contents: read
4646
id-token: write # OIDC trusted publishing: only this job mints the token

0 commit comments

Comments
 (0)