ReviewOS

also looking at this

craft-native/craft

chore(ci): pin zig-js at #876, the first revision that builds on Zig 1963

#106
Merged chris wants to merge chore/bump-zig-js-1963 into main
1 file +8 -2
11 days ago

main has been red on zig-core since #104 moved craft to 0.17.0-dev.1963. Every failure was inside zig-js at the old pin — that toolchain appends pointer_stability to ArrayList, and zig-js had 27 struct literals plus one JIT layout assertion that named the old shape.

zig-utils/zig-js#876 fixes it upstream — 53/53 checks on the 1818 their CI pins, and 2285 passed / 0 failed on 1963. This is the one-line pin.

Verified locally: craft's full zig build and zig build test with the JS runtime on, against this zig-js content, on 1963 — 153/153 steps, 2519 passed, 0 failed.

Unblocks integration, which has been skipping behind zig-core.

1 changed file on the files tab, with 0 review threads.

Sign in to comment on this pull request.