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

update for zig 0.13.0 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sethvincent
Copy link

Hi!

This pull request updates the codebase for zig 0.13.0.

std.json has changed considerably over the past couple years so with some of these changes I tried to keep a balance between using the new changes and keeping things simple.

The biggest examples of this are:

  • StringifyOptions changed shape a lot
  • the formatting of floats changed

The StringifyOptions change is straightforward: I just grabbed the old version of it and inlined it so that the code in main.zig could use the same struct fields. Updating the stringify writeStream code to what's in 0.13.0 might take quite some time 😬

About the formatting of floats: this is the only part of the pr that i have low confidence about, because it meant changing the expected values in the tests to what 0.13.0 outputs. I can make adjustments to the formatting if we find that my changes are incorrect.

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.

1 participant