-
Notifications
You must be signed in to change notification settings - Fork 46
Add support for roslyn client to mason.nvim #10
Comments
Is there something else, except for #9, that is blocking this? Or is it just a matter of making a PR in the mason-registry? |
It's just a matter of making the PR on the mason-registry right now. I have a branch of mason-registry that contains the necessary changes, just need to open the PR, but I haven't had time for this. I will try to prioritize it, but feel free to just fork and open the PR yourself. |
I'll leave your PR to you! I wouldn't be able to answer questions about it. When it is merged in mason, it would also be nice to get it into mason-lspconfig. I think a lot of people get their LSP setups from there. |
Are we just planning to make the language server from Microsoft installable with Mason? It would be nice to remove the need to install this plugin separately, and just use the mason-lspconfig flow? If I understand correctly, wouldn't that require Mason to install this project, as well as the language server? |
for the language server to work with mason-lspconfig this plugin should not exist. mason-lspconfig just uses lspconfig to setup the installed server. But in the case of roslyn there are a few reasons why it can't work with nvim-lspconfig and needs a separate plugin (this plugin). See this comment for more details. |
Arguably this is not a task to be completed on this repository but rather on mason.nvim. Creating this issue just for reference. Also relevant: #9
The text was updated successfully, but these errors were encountered: