Skip to content

Update net analyzers to 10.0#101

Merged
jryckman merged 5 commits intoni:mainfrom
yaqian256:users/yaqian/update-net10-analyzers
Dec 3, 2025
Merged

Update net analyzers to 10.0#101
jryckman merged 5 commits intoni:mainfrom
yaqian256:users/yaqian/update-net10-analyzers

Conversation

@yaqian256
Copy link
Contributor

@yaqian256 yaqian256 commented Nov 18, 2025

Justification

With the release of .NET 10 there are new analyzers we can use.

Implementation

  • Address the feedback from Update to .NET 8 #102
  • Update Microsoft.CodeAnalysis.NetAnalyzers to 10.0.100.
  • Update the default AnalysisLevel to 10.0 so that consumers using older .NET SDK will still get to use the new .NET 10 analyzers.

Testing

Built nupkg locally and confirmed new warnings are emitted in my projects when the analyzer package is updated.

@yaqian256 yaqian256 marked this pull request as ready for review December 2, 2025 02:24
@yaqian256 yaqian256 mentioned this pull request Dec 2, 2025
@yaqian256
Copy link
Contributor Author

NOTE: followed #100 to make this change.

@jryckman jryckman merged commit fb643b0 into ni:main Dec 3, 2025
1 check passed
jryckman added a commit that referenced this pull request Dec 3, 2025
* Initial plan

* Revert commit fb643b0: Revert analyzer versions and nuget configuration changes

Co-authored-by: jryckman <4111629+jryckman@users.noreply.github.com>

* Remove accidentally committed build artifacts and add .objs to gitignore

Co-authored-by: jryckman <4111629+jryckman@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jryckman <4111629+jryckman@users.noreply.github.com>
jryckman added a commit that referenced this pull request Dec 4, 2025
jryckman added a commit that referenced this pull request Dec 4, 2025
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