update the lsp type definition to comply the lsp spec 317 to make sure csharp lsp can work

9fans / acme-lsp

Language Server Protocol tools for the acme text editor

MIT License

193 stars 25 forks source link

update the lsp type definition to comply the lsp spec 317 to make sure csharp lsp can work #47

Closed ianzhang366 closed 1 year ago

ianzhang366 commented 2 years ago

Trying to make sure csharp lsp can work with acme-lsp together.

SInce the acme-lsp is sending response fields pre lsp-spec 317, but our acme-lsp's type is defined at 316. In this case modifying these fields

Signed-off-by: Ian Zhang izhang@redhat.com

fhs commented 1 year ago

We're now using the latest auto-generated LSP type definitions used by gopls: https://github.com/fhs/go-lsp-internal The csharp LSP server might work better now, but I haven't done any testing. Please open an issue if you encounter any incompatibilities.