Add Meson Muon extension - #4532
Conversation
MrSubidubi
left a comment
There was a problem hiding this comment.
Thanks for this!
Given that the existing extension is unmaintained, we would happily let you take over the existing extension ID and maintain that going forward.
Also/Alternatively, we could also fork the existing extension into the zed-extensions organization and make you a maintainer there.
I'd like to avoid having an abandoned extension and would prefer if we would just have one extension that is maintained going forward. Would you be interested in one of these solutions?
|
Hello @MrSubidubi both proposed solutions are fine by me, but I think it would be best for now to leave the older extension available for a little longer. The original extension uses MesonLSP as the backing LSP which is archived and no longer maintained but still very feature rich and working near perfectly on the newest meson files. My new extension uses the LSP mode of Muon as the backing LSP. This is a bit of a newer project and their focus is not solely on the LSP/analyzer side. I wanted to create this extension so it is easy to try out Muon even though it has a lot of features. The more people using it the better as it will also bring encouragement for it to be further developed. Muon is very actively maintained and clearly the future for Meson language servers. So because of this context, I think it might be better to have both available for now. What do you think? As far as I don't think there is a way to provide custom settings for extensions in Zed, correct? Otherwise I could support both LSPs in my new extension and replace the old one, letting users choose their preferred LSP. I have not seen the custom settings for extensions in the Zed docs but I might be wrong. Please provide a docs link in case I missed something. |
There was a problem hiding this comment.
Hey, sorry for the delay in review here, I unfortunately got sick and hence got not back to replying here yet.
In the meantime, the meson upstream repository was removed. I'd be happy to have you take over the meson extension ID and keep the repo in your personal account for the time being.
For that, pleaes
- update the ID of your extensions to
mesonboth in the repository as well as here. - add the extension at
0.5.0.
Does that sound good to you?
|
Sounds good! I will try to update my extension so users can switch between Muon LSP and mesonlsp and then come back here in a few days with the updated ID and version etc. |
|
Wonderful, thank you! |
4d85306 to
365202c
Compare
|
@MrSubidubi I updated everything as you mentioned. Changed the extension ID and and version. Please review the new PR (because this one got auto closed due to my force push when syncing). |
Add an extension to support the Meson build system after the existing Meson zed extension was no longer maintained and removed. Please view #4532 for the full context of adding this extension. @MrSubidubi
Add an extension to support the Meson build system after the existing Meson zed extension was no longer maintained and removed. Please view zed-industries#4532 for the full context of adding this extension. @MrSubidubi
Add an extension to support the Meson build system after the existing Meson zed extension was no longer maintained and removed. Please view zed-industries#4532 for the full context of adding this extension. @MrSubidubi
Add an alternative extension that adds language support for
meson.buildfiles with syntax highlighting and theMuon LSPto replaceMesonLSP. Submitting this extension since the currentextensions/mesonextension is unmaintained and was archived. The existingextensions/mesonextension usesMesonLSPfor the language server but this project is also unmaintained and archived. This extension uses the LSP feature ofMuon.