Skip to content

[Feature] Support csharp-ls #22

@AdeAttwood

Description

@AdeAttwood

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions