-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Do not export IDE0079 (remove unnecessary suppressions) analyzer in C… #47944
Conversation
@jmarolf Will the CodeStyle assemblies from master eventually flow into .NET5 SDK? This is a fix to the CodeStyle package itself. |
...eUnnecessarySuppressions/VisualBasicRemoveUnnecessaryPragmaSuppressionsDiagnosticAnalyzer.vb
Outdated
Show resolved
Hide resolved
...eUnnecessarySuppressions/VisualBasicRemoveUnnecessaryPragmaSuppressionsDiagnosticAnalyzer.vb
Outdated
Show resolved
Hide resolved
@mavasani I can update the sdk to include this change. Does this need to do into VS 16.8? If so I believe it will need to target a different branch |
@jmarolf I thought master is still targeting 16.8 P4. @vatsalyaagrawal @jinujoseph to confirm. |
Thanks @jmarolf! |
…odeStyle layer Fixes dotnet#47942 This is a special IDE-only analyzer that should not be exported in the CodeStyle NuGet package. We still want to retain the code in the shared analyzer layer to allow different analyzer hosts to be able to explicitly instantiate it and enforce it on command line from the CodeStyle package.
ef623dc
to
315c5bf
Compare
Hello @mavasani! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto-approval
…odeStyle layer
Fixes #47942
This is a special IDE-only analyzer that should not be exported in the CodeStyle NuGet package. We still want to retain the code in the shared analyzer layer to allow different analyzer hosts to be able to explicitly instantiate it and enforce it on command line from the CodeStyle package.