ReviewOS

also looking at this

craft-native/craft

ci: pin the pantry action to a commit

#45
Closed chris wants to merge ci/pin-pantry-action into main
7 files +35 -15
.github/workflows/README.mdmodified+20-0
Changes to .github/workflows/README.md
@@ -11,3 +11,23 @@ This folder contains the following GitHub Actions:
1111
1212[CI]: ./workflows/ci.yml
1313[Release]: ./workflows/release.yml
14
15## Pinned actions
16
17`pantry-pm/pantry/packages/action` is pinned to a commit, not to `@main`.
18
19`@main` means an unrelated push to another repository decides whether every
20job here runs, and on 2026-08-20 that happened twice in one night: first the
21action stopped shipping its built `dist/index.js`, so every job failed at step
22one with `File not found`; then, after that was fixed upstream, the setup step
23began hanging — every job on both Linux and macOS sat on **Setup Pantry** until
24GitHub killed it at the six-hour ceiling.
25
26The pin is `ef37fca8440cdb483fb355fefa5ae3cc6e669a69` (pantry 0.11.36), the
27revision the last fully green run on `main` used.
28
29To bump it: change the SHA everywhere it appears under `.github/workflows/`,
30in one commit, and let CI prove the new revision before it lands. The same
31reasoning applies to the first-party Zig dependencies — see
32[`.github/actions/first-party-zig-deps/pins.env`](../actions/first-party-zig-deps/pins.env).
33