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

Filter diagnostics lint level in languages.toml #1007

Closed
jbeaurivage opened this issue Nov 7, 2021 · 0 comments · Fixed by #1325
Closed

Filter diagnostics lint level in languages.toml #1007

jbeaurivage opened this issue Nov 7, 2021 · 0 comments · Fixed by #1325
Labels
A-helix-term Area: Helix term improvements A-language-server Area: Language server client C-enhancement Category: Improvements

Comments

@jbeaurivage
Copy link

Describe your feature request

It would be nice to be able to filter the lint level of diagnostics being highlighted. For instance, in this screenshot of Rust code, rust-analyzer emits info diagnostics for non-expanded proc-macros, which is mostly just noise.

Screenshot_20211107_091930

@jbeaurivage jbeaurivage added the C-enhancement Category: Improvements label Nov 7, 2021
@kirawi kirawi added the A-helix-term Area: Helix term improvements label Nov 7, 2021
@pickfire pickfire added the A-language-server Area: Language server client label Nov 17, 2021
archseer pushed a commit that referenced this issue Dec 25, 2021
* feat(lsp): configurable diagnostic severity

Allow severity of diagnostic messages to be configured.
E.g. allow turning of Hint level diagnostics.

Fixes: #1007

* Use language_config() method

* Add documentation for diagnostic_severity

* Use unreachable for unknown severity level

* fix: documentation for diagnostic_severity config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements A-language-server Area: Language server client C-enhancement Category: Improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants