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

feat: add build.zig.zon #395

Merged
merged 2 commits into from
Feb 9, 2024
Merged

feat: add build.zig.zon #395

merged 2 commits into from
Feb 9, 2024

Conversation

pseudocc
Copy link
Contributor

@alex-courtis
Copy link
Member

Seti icon looks good.

Should we be using simply zig.zon rather than build? Is it a general zig file extension?

@pseudocc
Copy link
Contributor Author

pseudocc commented Feb 7, 2024

By far, Zig(v0.11.0) does not provide a simple way to parse zon file, even parsing build.zig.zon is kinda hard coded.

To me, both "build.zig.zon" and "*.zig.zon" are good. Sorry for the late reply, I am enjoying Chinese New Year now.

@alex-courtis
Copy link
Member

By far, Zig(v0.11.0) does not provide a simple way to parse zon file, even parsing build.zig.zon is kinda hard coded.

To me, both "build.zig.zon" and "*.zig.zon" are good. Sorry for the late reply, I am enjoying Chinese New Year now.

Understood. Hardcoded build it is.

GXFC

@alex-courtis alex-courtis merged commit 55e956f into nvim-tree:master Feb 9, 2024
4 checks passed
@hornyjailfish
Copy link

broken! remove spaces inside hl name string

@zyriab
Copy link
Contributor

zyriab commented Feb 9, 2024

Hey guys, this broke my Neovim :(
image

@zyriab
Copy link
Contributor

zyriab commented Feb 9, 2024

I opened #397 :)

@rafamancan
Copy link

I opened #397 :)

I made your modification on my config and works fine!
Thx.

@alex-courtis
Copy link
Member

Many thanks for the quick fix @zyriab

@zyriab
Copy link
Contributor

zyriab commented Feb 11, 2024

Many thanks for the quick fix @zyriab

Cheers :)

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