Commit 5796ff6
committed
ci: adopt the setup-pnpm composite action at all 16 remaining Corepack sites
`corepack enable` only writes shims; the pnpm tarball is fetched from
registry.npmjs.org on the first pnpm invocation in the job. Every such site
is an independent chance for a network flake to fail a job for a reason that
has nothing to do with the code under test.
The setup-pnpm composite action removes that call by restoring the Corepack
store from the actions cache. It was introduced for ci.yml, which now has
zero Corepack sites. This applies it to the ten workflows that still had one.
Counts across .github/workflows/ (before -> after):
corepack enable occurrences 16 -> 0 (10 files -> 0)
uses: ./.github/actions/setup-pnpm 7 -> 23 (+16, the matching amount)
name: Verify pnpm version 8 -> 0 (folded into the composite)
uses: actions/setup-node@ 32 -> 32 (unchanged, deliberately)
Every caller keeps its own actions/setup-node step with its literal
node-version pin. scripts/check-node-version.mjs scans .github/workflows/*.yml
only and reports how many setup-node steps it audited; moving those steps into
the composite would drop them from its census while it still printed OK. It
still reports 32 setup-node steps across 26 workflows.
The two sites whose Corepack step was conditional keep their condition
verbatim on the composite step, and stay in position:
pr-automation.yml changeset-check (skip label / diff-base guard)
release.yml release-integrity (deferred to the repair path only)
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015ahemw8RcTgqtxrj15PEZx1 parent 2a6122b commit 5796ff6
10 files changed
Lines changed: 32 additions & 56 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
32 | | - | |
| 31 | + | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
339 | 339 | | |
340 | 340 | | |
341 | 341 | | |
342 | | - | |
343 | | - | |
344 | | - | |
345 | | - | |
346 | | - | |
| 342 | + | |
| 343 | + | |
347 | 344 | | |
348 | 345 | | |
349 | 346 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
60 | | - | |
| 59 | + | |
| 60 | + | |
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
| |||
2496 | 2496 | | |
2497 | 2497 | | |
2498 | 2498 | | |
2499 | | - | |
2500 | | - | |
2501 | | - | |
2502 | | - | |
2503 | | - | |
| 2499 | + | |
| 2500 | + | |
2504 | 2501 | | |
2505 | 2502 | | |
2506 | 2503 | | |
| |||
3072 | 3069 | | |
3073 | 3070 | | |
3074 | 3071 | | |
3075 | | - | |
3076 | | - | |
3077 | | - | |
3078 | | - | |
3079 | | - | |
| 3072 | + | |
| 3073 | + | |
3080 | 3074 | | |
3081 | 3075 | | |
3082 | 3076 | | |
| |||
3249 | 3243 | | |
3250 | 3244 | | |
3251 | 3245 | | |
3252 | | - | |
3253 | | - | |
3254 | | - | |
3255 | | - | |
3256 | | - | |
| 3246 | + | |
| 3247 | + | |
3257 | 3248 | | |
3258 | 3249 | | |
3259 | 3250 | | |
| |||
3443 | 3434 | | |
3444 | 3435 | | |
3445 | 3436 | | |
3446 | | - | |
3447 | | - | |
3448 | | - | |
3449 | | - | |
3450 | | - | |
| 3437 | + | |
| 3438 | + | |
3451 | 3439 | | |
3452 | 3440 | | |
3453 | 3441 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
423 | 423 | | |
424 | 424 | | |
425 | 425 | | |
426 | | - | |
| 426 | + | |
427 | 427 | | |
428 | 428 | | |
429 | 429 | | |
430 | | - | |
| 430 | + | |
431 | 431 | | |
432 | 432 | | |
433 | 433 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
191 | 191 | | |
192 | 192 | | |
193 | 193 | | |
194 | | - | |
195 | | - | |
| 194 | + | |
| 195 | + | |
196 | 196 | | |
197 | 197 | | |
198 | 198 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
274 | 274 | | |
275 | 275 | | |
276 | 276 | | |
277 | | - | |
278 | | - | |
279 | | - | |
280 | | - | |
281 | | - | |
| 277 | + | |
| 278 | + | |
282 | 279 | | |
283 | 280 | | |
284 | 281 | | |
| |||
507 | 504 | | |
508 | 505 | | |
509 | 506 | | |
510 | | - | |
| 507 | + | |
511 | 508 | | |
512 | | - | |
| 509 | + | |
513 | 510 | | |
514 | 511 | | |
515 | 512 | | |
| |||
670 | 667 | | |
671 | 668 | | |
672 | 669 | | |
673 | | - | |
674 | | - | |
675 | | - | |
676 | | - | |
677 | | - | |
| 670 | + | |
| 671 | + | |
678 | 672 | | |
679 | 673 | | |
680 | 674 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
57 | | - | |
58 | | - | |
| 57 | + | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
55 | | - | |
56 | | - | |
| 55 | + | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
48 | | - | |
49 | | - | |
| 48 | + | |
| 49 | + | |
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
| 42 | + | |
| 43 | + | |
47 | 44 | | |
48 | 45 | | |
49 | 46 | | |
| |||
0 commit comments