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

Use same severity levels as Roslyn analyzers #59

Open
cmeeren opened this issue Jan 2, 2023 · 0 comments
Open

Use same severity levels as Roslyn analyzers #59

cmeeren opened this issue Jan 2, 2023 · 0 comments

Comments

@cmeeren
Copy link
Contributor

cmeeren commented Jan 2, 2023

The Roslyn severity levels are (showing the .editorconfig values below):

  • error
  • warning
  • suggestion ("Info" in VS)
  • silent ("Hidden in VS; as far as I know, this will show quick-fixes in the IDE, but hide the squiggles/message)
  • none (completely disabled, including quick-fixes)
  • default (the default set by the analyzer for that rule)

We could consider using the same values. Currently, we only have Error, Warning, and Info (which could be renamed to "Suggestion"; I find that a more descriptive name, and it fits the Roslyn editorconfig values).

Note that Default of course is just for configuration; analyzers should not be able to set this.

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

1 participant