Skip to content

Move some more rules to System.Runtime.Analyzers - #925

Merged
srivatsn merged 3 commits into
dotnet:masterfrom
srivatsn:sysruntime
Mar 4, 2015
Merged

Move some more rules to System.Runtime.Analyzers#925
srivatsn merged 3 commits into
dotnet:masterfrom
srivatsn:sysruntime

Conversation

@srivatsn

Copy link
Copy Markdown
Contributor

CA2231 - this is very similar to CA1036. So refactored them a bit to share code. Simplified the fixer with SyntaxGenerator.
CA1014, CA1016, CA1017 are all analyzers for assembly attributes and they don't have fixers because we don't have fixes for project-level issues.

…es are all about assembly level attributes and dont' have fixers because we don't have lightbulbs in the error list yet (for issues that don't have a location).
@srivatsn

Copy link
Copy Markdown
Contributor Author

@mavasani @heejaechang @tmeschter @shyamnamboodiripad @JohnHamby - can a couple of you take a look?

@srivatsn

srivatsn commented Mar 3, 2015

Copy link
Copy Markdown
Contributor Author

Ping

@mavasani

mavasani commented Mar 3, 2015

Copy link
Copy Markdown
Contributor

👍

srivatsn added a commit that referenced this pull request Mar 4, 2015
Move some more rules to System.Runtime.Analyzers
@srivatsn
srivatsn merged commit 3e095f9 into dotnet:master Mar 4, 2015
dibarbet pushed a commit that referenced this pull request Aug 18, 2026
Update build to an image that includes Visual Studio 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants