Skip to content

Include Roslynator.Analyzers and Roslynator.CodeFixes with more strict default severity levels #83

Description

@ascott18

Introduce https://github.com/JosefPihrt/Roslynator, which has 500+ analyzers, and also a large number of refactorings.

Lists of each:

http://pihrt.net/Roslynator/Analyzers
http://pihrt.net/Roslynator/Refactorings

Here's a very preliminary set of some recommendations for default severities. This is by no means exhaustive and is only the set that I came up with upon looking at the Info level severity issues from a single project I'm working on at the moment.

For some of these that aren't already covered, corresponding rules should be added to the C# standards.

# RCS1001: Add braces (when expression spans over multiple lines).dotnet_diagnostic.RCS1001.severity = error# RCS1003: Add braces to if-else (when expression spans over multiple lines).dotnet_diagnostic.RCS1003.severity = error# RCS1007: Add bracesdotnet_diagnostic.RCS1007.severity = error# RCS1044: Remove original exception from throw statementdotnet_diagnostic.RCS1044.severity = error# RCS1123: Add parentheses according to operator precedence.dotnet_diagnostic.RCS1123.severity = error# RCS1037: Remove trailing white-space.dotnet_diagnostic.RCS1037.severity = none# RCS1036: Remove redundant empty line.dotnet_diagnostic.RCS1036.severity = none# RCS1170: Use read-only auto-implemented property.dotnet_diagnostic.RCS1170.severity = warning# RCS1213: Remove unused member declaration.dotnet_diagnostic.RCS1213.severity = warning

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    .editorconfigChanges here are to the editorconfigHacktoberfestdependenciesPull requests that update a dependency fileenhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions