roslyn icon indicating copy to clipboard operation
roslyn copied to clipboard

Fix all warning and errors set in Editorconfig does not do what it promises

Open vsfeedback opened this issue 1 year ago • 1 comments

This issue has been moved from a ticket on Developer Community.


I've setup an .editorconfig in the solution, that highlights violations to the team standard as errors. I want the developers to be able to autofix most of these issue using Clean Up tool.

However, the existing fixer "Fix all warning and errors set in Editorconfig":

  • Fixes issues that are not described in editorconfig (defaults)
  • Fixes issues that are of severity Suggestion and Silent
  • There is no way to ignore warnings without overriding or suppressing them

As such, I think that this fixer does not deliver what it promises.

Of course, I can override the severity of some warnings to none, but It's not a good solution because I want the developers to see them and make conscious decisions around them.

What I propose

  1. Rename existing fixer to what it actually does: "Fix all issues (including settings in Editorconfig)"
  2. Create new Fixer that fixes ONLY errors and warnings
  3. Create new Fixer that fixes ONLY errors

Original Comments

Feedback Bot on 9/17/2023, 07:56 PM:

(private comment, text removed)

Mika Dumont [MSFT] on 12/19/2023, 07:11 AM:

(private comment, text removed)

Feedback Bot on 12/26/2023, 09:28 PM:

(private comment, text removed)

vsfeedback avatar Mar 14 '24 17:03 vsfeedback

It would be great to have a fix for errors only or a fix that can be configured with a list of rules

bart-medfar avatar Jun 27 '24 03:06 bart-medfar

Duplicate of https://github.com/dotnet/roslyn/issues/66905?

parched avatar Sep 09 '24 03:09 parched