Controlling workspace/diagnostic/refresh #18424
Labels
A-lsp
LSP conformance issues and missing features
Broken Window
Bugs / technical debt to be addressed immediately
C-bug
Category: bug
E-easy
rust-analyzer version: rust-analyzer 0.3.2162-standalone (3b3a87f 2024-10-27)
rustc version: rustc 1.81.0 (eeb90cda1 2024-09-04)
editor or extension: emacs lsp-mode
relevant settings: (eg. client settings, or environment variables like
CARGO
,RUSTC
,RUSTUP_HOME
orCARGO_HOME
)repository link (if public, optional): rust-analyzer)
When pull diagnostics were introduced in LSP there was no specific client configuration to enable them, since it is up to the client to pull them.
Pull diagnostics are not currently implemented emacs lsp-mode.
This week's RA release enabled pull diagnostics, together with
workspace/diagnostic/refresh
, which originates from the server. This server request should be gated bybut is currently gated on the mere presence of the
textDocument.diagnostics
client config. (viarust-analyzer/crates/rust-analyzer/src/main_loop.rs
Line 420 in 3b3a87f
So this decision should be made based on
workspace.diagnostice.refreshSupport
instead.Related: emacs-lsp/lsp-mode#4570
The text was updated successfully, but these errors were encountered: