diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-06-17 12:01:37 +0100 |
---|---|---|
committer | GitHub <[email protected]> | 2020-06-17 12:01:37 +0100 |
commit | 931f3173992df6ac6b728fa9fa9a94d15781027e (patch) | |
tree | a0f27ea80505d451673e3f035c7a1e8d5b4bc97c /crates/ra_hir_def/src/path/lower.rs | |
parent | 09c5cfedff0542d79ae0575955095aa98cc41c16 (diff) | |
parent | 2fa0b20ce05a1d6b8f9bf0f0d9f1dfb15b84fb9d (diff) | |
parent | 8ff91cf6b649ea241e886726df91d9bcb6b6c7cb (diff) | |
parent | 656e95211edf26bd82bd66f53ca3bcdfb7296e81 (diff) |
Merge #4913 #4915 #4916
4913: Remove debugging code for incremental sync r=matklad a=lnicola
4915: Inspect markdown code fences to determine whether to apply syntax highlighting r=matklad a=ltentrup
Fixes #4904
4916: Warnings as hint or info r=matklad a=GabbeV
Fixes #4229
This PR is my second attempt at providing a solution to the above issue. My last PR(#4721) had to be rolled back(#4862) due to it overriding behavior many users expected. This PR solves a broader problem while trying to minimize surprises for the users.
### Problem description
The underlying problem this PR tries to solve is the mismatch between [Rustc lint levels](https://doc.rust-lang.org/rustc/lints/levels.html) and [LSP diagnostic severity](https://microsoft.github.io/language-server-protocol/specification#diagnostic). Rustc currently doesn't have a lint level less severe than warning forcing the user to disable warnings if they think they get to noisy. LSP however provides two severitys below warning, information and hint. This allows editors like VSCode to provide more fine grained control over how prominently to show different diagnostics.
Info severity shows a blue squiggly underline in code and can be filtered separately from errors and warnings in the problems panel.
![image](https://user-images.githubusercontent.com/13839236/84830640-0bb8d900-b02a-11ea-9e2f-0561b0e8f1ef.png)
![image](https://user-images.githubusercontent.com/13839236/84826931-ffca1880-b023-11ea-8080-5e5b91a6ac0d.png)
Hint severity doesn't show up in the problems panel at all and only show three dots under the affected code or just faded text if the diagnostic also has the unnecessary tag.
![image](https://user-images.githubusercontent.com/13839236/84827165-55062a00-b024-11ea-8bd6-bdbf1217c4c5.png)
### Solution
The solution provided by this PR allows the user to configure lists of of warnings to report as info severity and hint severity respectively. I purposefully only convert warnings and not errors as i believe it's a good idea to have the editor show the same severity as the compiler as much as possible.
![image](https://user-images.githubusercontent.com/13839236/84829609-50437500-b028-11ea-80a8-1bbd05680ba7.png)
### Open questions
#### Discoverability
How do we teach this to new and existing users? Should a section be added to the user manual? If so where and what should it say?
#### Defaults
Other languages such as TypeScript report unused code as hint by default. Should rust-analyzer similarly report some problems as hint/info by default?
Co-authored-by: Laurențiu Nicola <[email protected]>
Co-authored-by: Leander Tentrup <[email protected]>
Co-authored-by: Gabriel Valfridsson <[email protected]>