Replies: 1 comment 2 replies
-
Yes. This is called "pull diagnostics" or something.
Yes, it would. In fact, this should be closer to Emacs's model of getting diagnostics for other languages. However this is an very tricky part of Eglot and I would like to not break it or over-complexify it. So a carefully tested and well-designed implementation must be done. Please send email to [email protected] to open a bug report with this feature request, include these messages as context, and remember to CC me. Thanks. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I'm not sure how common this is across other LSP servers, but
ruby-lsp
does not usetextDocument/publishDiagnostics
to publish diagnostics - it only uses that event to clear them out on document close.Instead, their VSCode plugin relies on polling for diagnostics in an interval with
textDocument/diagnostic
. a quick look through the Eglot source and I don't see evidence of a similar mechanism. is this correct? in theruby-lsp
wiki it seems like the Neovim LSP client also doesn't support this and so there is some additional code someone has created for this purpose.if this currently isn't a feature of Eglot, would it make sense to entertain as a new feature? are there any other known LSP servers that rely on polling for diagnostics like this?
Beta Was this translation helpful? Give feedback.
All reactions