Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: upgrade zig to 0.14.0 #17820

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

chore: upgrade zig to 0.14.0 #17820

wants to merge 18 commits into from

Conversation

190n
Copy link
Contributor

@190n 190n commented Mar 1, 2025

Major causes of changes:

  • std.mem.Allocator vtable changed (alignment is an enum instead of a u8; remap function added)
  • std.ArrayList.pop changed to return an optional, and std.ArrayList.popOrNull deleted
  • runtime page size
  • minor zig fmt changes
  • std.ArrayList initial capacity changed (the places that broke have been changed to not rely on the capacity)

@robobun
Copy link

robobun commented Mar 1, 2025

Updated 3:35 PM PT - Mar 7th, 2025

@190n, your commit f3c19ff has 1 failures in Build #12849:


🧪   try this PR locally:

bunx bun-pr 17820

@pfgithub
Copy link
Contributor

pfgithub commented Mar 5, 2025

zls is broken, this run will probably fix it https://github.com/oven-sh/zig/actions/runs/13686626041/job/38271434392

@190n 190n marked this pull request as ready for review March 6, 2025 20:06
@190n 190n changed the title chore: upgrade zig chore: upgrade zig to 0.14.0 Mar 6, 2025
@190n 190n requested review from a team and cirospaciari and removed request for a team March 6, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants