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/workflows/binary-size.ymlmodified+2-2
Changes to .github/workflows/binary-size.yml
@@ -71,7 +71,7 @@ jobs:
7171 - name: Build release binary
7272 working-directory: packages/zig
7373 run: |
74 eval "$(pantry env | sed -n '/^export /,$p')" && zig build -Doptimize=ReleaseSafe
74 zig build -Doptimize=ReleaseSafe
7575
7676 - name: Measure binary size
7777 id: size
@@ -123,7 +123,7 @@ jobs:
123123 git checkout origin/main -- packages/zig
124124
125125 cd packages/zig
126 eval "$(pantry env | sed -n '/^export /,$p')" && zig build -Doptimize=ReleaseSafe
126 zig build -Doptimize=ReleaseSafe
127127
128128 if [ -f "zig-out/bin/craft" ]; then
129129 BASELINE=$(wc -c < "zig-out/bin/craft" | tr -d ' ')