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

Proposing new rule 'Remove unnecessary suppression' same as IDE0079 #516

Open
maettu-this opened this issue Aug 6, 2023 · 1 comment
Open

Comments

@maettu-this
Copy link

IDE0079: Remove unnecessary suppression works fine for IDE.... rules, but not for other analyzers. Proposing to add a new IDISP.... rule 'Remove unnecessary suppression'.

Motivation: Clean code, minimum of suppressions.

Such new rule could be considered too intrusive. Still, it could be added as "opt-in" rule, i.e. keep disabled by default at first and only later default to enbled once the new rule has been adopted by the majority of users.

@maettu-this
Copy link
Author

In the meantime I got told at DotNetAnalyzers/StyleCopAnalyzers#3685 that IDE0079 is supposed to include other analyzers but doesn’t fully work yet. If IDE0079 indeed will cover the IDisposableAnalyzers, this proposal would be obsolete.

Feel free to keep open until IDE0079 got fixed, or already close it now.

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