Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug (Neovim): Diagnostics not working #63

Open
m0tholith opened this issue Jul 14, 2024 · 1 comment
Open

Bug (Neovim): Diagnostics not working #63

m0tholith opened this issue Jul 14, 2024 · 1 comment

Comments

@m0tholith
Copy link

Diagnostics don't really work on Neovim. Completion, documentation and everything else works well, attached is my LSP logfile:
lsp.log

@nolanderc
Copy link
Owner

nolanderc commented Jul 31, 2024

Yeah, I never got around to publishing diagnostics to the client, even though we collect them during parsing.

Diagnostics are published as a LSP notification from the server to the client, and so far this isn't implemented, and thus would need some work (altough, likely not that much).

With that implemented, we could send notifications upon receiving a textDocument/didChange or textDocument/didOpen request from the client. But we probably need some mechanism for queuing notifications, so that we don't send the notification until after having responded to the initial request.

I probably won't have the time to work on this for a while, but PRs are always welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants