Skip to content

Updating zig build for 0.16.0 - #147

Merged
dougbinks merged 3 commits into
dougbinks:devfrom
Honeybunch:master
Aug 6, 2026
Merged

dougbinks merged 3 commits into
dougbinks:devfrom
Honeybunch:master

Conversation

@Honeybunch

Copy link
Copy Markdown
Contributor

Thanks for adding a build.zig script! I love using enkiTS from zig 😄

I had to make some small fixes for 0.16.0 which released back in April. Figured others could use this as well so enkiTS just works with the latest zig release.

@dougbinks

Copy link
Copy Markdown
Owner

I'm currently overloaded with work but will hopefully get around to this shortly.

@dougbinks
dougbinks self-requested a review July 24, 2026 08:05
@dougbinks dougbinks self-assigned this Jul 24, 2026
@dougbinks

Copy link
Copy Markdown
Owner

Since @Avokadoen created the zig build in #143 and I don't use zig, I'd like them to review this PR if possible. For example I'm not sure if the minimum version should be the current version or if there is a backwards compatible way of making the changes.

@Avokadoen

Copy link
Copy Markdown
Contributor

I can test it and take a look tomorrow!

@Avokadoen

Copy link
Copy Markdown
Contributor

For the record. Glancing over, the change is looking good. I don't think there is much value in keeping the minimum to the lowest either way because users of older zig can stick to older enkits. I can do some actual testing tomorrow when I have access to a PC.

@dougbinks

Copy link
Copy Markdown
Owner

Many thanks @Avokadoen

@Avokadoen

Avokadoen commented Aug 6, 2026 •

Copy link
Copy Markdown
Contributor

@Honeybunch .github\workflows\build.yml:71 and .github\workflows\build.yml:142 needs to be updated to use version 0.16.0

@Honeybunch

Copy link
Copy Markdown
Contributor Author

Makes sense! I can do that later this morning

@Avokadoen

Copy link
Copy Markdown
Contributor

With the exception of my comment to update CI, this lgtm!

Thanks @Honeybunch!

Do what you will with this information @dougbinks! :D

@dougbinks

Copy link
Copy Markdown
Owner

Many thanks all, once the build.yml change is in I will merge this.

@Honeybunch

Copy link
Copy Markdown
Contributor Author

Updated build.yml 😄

@dougbinks
dougbinks changed the base branch from master to dev August 6, 2026 20:29
@dougbinks
dougbinks merged commit ccd4e8c into dougbinks:dev Aug 6, 2026
7 of 12 checks passed
@dougbinks

Copy link
Copy Markdown
Owner

Merged, many thanks.

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.

3 participants