-
Notifications
You must be signed in to change notification settings - Fork 22
Description
Add support for csharp-ls in zed for csharp files. I have made a working implementation in my fork AdeAttwood@71e90fe.
I have piggybacked off the abstraction done in the Roslyn LSP following the same pattern to add the new language server.
I have disabled omnisharp in my settings however, I'm not sure if this is needed.
"languages": {
"CSharp": {
"language_servers": ["csharp-ls", "!omnisharp"]
}
}The main questions for me are:
Is this something you would like to accept?
Is not installing the lsp OK?
All the other LSP's seem to have an install / update process. I did find some, like the gRPC one, that error like this one with a message to install the LSP. I am new to zed and could not see a way to run commands to get the LSP installed. If this is something you want, a nudge in the right direction would be helpful.
If this is a change you want when do you want a PR
I have held off for now because this builds on top of #11, I have only made one commit and diff looks out of scope. Let me know, happy to make a PR whenenver.