zigtools / zls-vscode

Deprecated, please use https://github.com/ziglang/vscode-zig
https://github.com/ziglang/vscode-zig
MIT License
85 stars 15 forks source link

Become totally independent from the official Zig extension #66

Open SuperAuguste opened 1 year ago

SuperAuguste commented 1 year ago

I think that's the way. More generally, I believe there shouldn't be separate zig and zig-zls vscode extensions, there should be a single extension that handles everything zig-related. From the user's perspective, an important part of UX is to be able to install a single thing, and trust the authors of the thing to correctly assemble a holistic user experience from the underlying tools.

To clarify, this integration would not preclude someone from shipping an alternative view of how it all should look like: the benefit is in integration, not in monopoly.

Originally posted by @matklad in https://github.com/zigtools/zls-vscode/issues/57#issuecomment-1420701033

I've let this stew a bit and I definitely agree with matklad's point of view, especially after reading this tweet. ZLS is an extremely helpful tool for beginners and everyone else and it's a shame that people choose the official Zig extension over this for various different reasons, from lack of certain features, but also a branding and trust issue. That shouldn't be the case.

matklad commented 1 year ago

Cc https://github.com/ziglang/zig/issues/10204

Create a new VSCode extension that does both syntax as well as ZLS. Currently there are 2 separate extensions that you have to install. Deprecate those two extensions in favor of the One Extension To Rule Them All.

SuperAuguste commented 1 year ago

I'll hit up Vexu and see if we can perhaps work together on this. I created this issue thinking that the opposite sentiment was true.

SuperAuguste commented 1 year ago

So total 180, we're going to try to integrate both extensions into one, thanks for pointing that out @matklad :)