Skip to content

Update roslyn-analyzers to 3.3.0-beta1.20262.5#45619

Merged
sharwell merged 3 commits intodotnet:masterfrom
sharwell:update-analyzers
Jul 7, 2020
Merged

Update roslyn-analyzers to 3.3.0-beta1.20262.5#45619
sharwell merged 3 commits intodotnet:masterfrom
sharwell:update-analyzers

Conversation

@sharwell
Copy link
Contributor

@sharwell sharwell commented Jul 2, 2020

No description provided.

@sharwell sharwell requested review from a team as code owners July 2, 2020 19:09
@sharwell
Copy link
Contributor Author

sharwell commented Jul 7, 2020

@mavasani How can I include 8a017ae without breaking source build? We end up in a situation where one of our analyzer packages is required for a clean source build.

/cc @jaredpar

@sharwell sharwell force-pushed the update-analyzers branch from 640a094 to cbabf49 Compare July 7, 2020 15:33
sharwell added 3 commits July 7, 2020 08:39
This suppressor applies to Nullable Reference Types (NRT), which is
problematic because source build has NRT enabled but analyzers are
disabled.
@sharwell sharwell force-pushed the update-analyzers branch from cbabf49 to 704d1ce Compare July 7, 2020 15:40
@sharwell sharwell merged commit e793cea into dotnet:master Jul 7, 2020
@ghost ghost added this to the Next milestone Jul 7, 2020
@sharwell sharwell deleted the update-analyzers branch July 7, 2020 20:20
@JoeRobich JoeRobich modified the milestones: Next, 16.8.P1 Jul 20, 2020
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

Successfully merging this pull request may close these issues.

3 participants