Skip to content

Refactor CA1305 to compare symbols#6356

Merged
mavasani merged 2 commits into
dotnet:mainfrom
Youssef1313:ca1305
Dec 21, 2022
Merged

Refactor CA1305 to compare symbols#6356
mavasani merged 2 commits into
dotnet:mainfrom
Youssef1313:ca1305

Conversation

@Youssef1313

@Youssef1313 Youssef1313 commented Dec 15, 2022

Copy link
Copy Markdown
Member

Fixes #6052

@Youssef1313 Youssef1313 requested a review from a team as a code owner December 15, 2022 11:23
@codecov

codecov Bot commented Dec 15, 2022

Copy link
Copy Markdown

Codecov Report

Merging #6356 (6edc75e) into main (37c910d) will increase coverage by 0.00%.
The diff coverage is 87.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6356   +/-   ##
=======================================
  Coverage   96.08%   96.08%           
=======================================
  Files        1362     1360    -2     
  Lines      315490   315500   +10     
  Branches    10187    10186    -1     
=======================================
+ Hits       303137   303148   +11     
+ Misses       9920     9918    -2     
- Partials     2433     2434    +1     

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.

Cleanup CA1305 analyzer to correctly compare method symbols

2 participants