zls icon indicating copy to clipboard operation
zls copied to clipboard

Update Zig master `@branchHint`

Open mochalins opened this issue 1 year ago • 1 comments

mochalins avatar Aug 27 '24 12:08 mochalins

Minimum Zig version not available to CI yet, must re-run once available. Tests pass locally on Linux x86_64. Tests fail for x86_64 Windows through Wine, for what appear to be Wine-specific reasons (hundreds of fixme and XXX not implemented yet errors logged during test).

mochalins avatar Aug 27 '24 12:08 mochalins

there are some more breaking changes on master, https://github.com/ziglang/zig/pull/21225 ~~and possibly https://github.com/ziglang/zig/pull/21231 eventually (havent checked if this one affects zls)~~(it doesnt). could maybe do them all here so there arent multiple prs trying to update?

xdBronch avatar Aug 29 '24 18:08 xdBronch

breaking changes are my passion

mlugg avatar Aug 29 '24 18:08 mlugg

there are some more breaking changes on master, https://github.com/ziglang/zig/pull/21225 and possibly https://github.com/ziglang/zig/pull/21231 eventually (havent checked if this one affects zls). could maybe do them all here so there arent multiple prs trying to update?

Sure, will do!

mochalins avatar Aug 29 '24 21:08 mochalins

Still waiting for this fix to be merged.

devraymondsh avatar Aug 31 '24 12:08 devraymondsh

Updated with all tests passing on latest master (0.14.0-dev.1392+37df6ba86) on x86_64 Linux. Depends on https://github.com/zigtools/zig-lsp-codegen/pull/18, build.zig.zon must be updated after it is merged.

mochalins avatar Aug 31 '24 15:08 mochalins