ReviewOS

also looking at this

craft-native/craft

fix(ci): stop evaluating `pantry env`, which sometimes re-installs JS deps mid-job

#107
Merged glennmichael123 wants to merge fix/ci-drop-pantry-env into main
6 files +16 -17
.github/CONTRIBUTING.mdmodified+1-1
Changes to .github/CONTRIBUTING.md
@@ -6,7 +6,7 @@ Thank you for your interest in contributing to Craft! This document provides gui
66
77### Prerequisites
88
9- **Pantry-managed stable Zig**: run `eval "$(pantry env | sed -n '/^export /,$p')"`, then invoke `zig`
9- **Pantry-managed stable Zig**: run `eval "$(pantry env | sed -n '/^export /,$p')"`, then invoke `zig` (locally only — CI does not: the pantry action already puts `zig` on PATH, and `pantry env` runs a workspace setup that re-installs JS deps with its own `--linker`, rewriting `node_modules` mid-job)
1010- **Bun**: Install from [bun.sh](https://bun.sh)
1111
1212#### Platform-specific Dependencies