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
Fixes CA1854 replacing unrelated dictionaries with matching signatures #6387
Conversation
Changed Analyzer to first find ContainsKey and than find all usages with the same unchanged key and dictionary reference in the guarded path. Fixes: dotnet#6256
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #6387 +/- ##
==========================================
- Coverage 96.41% 96.40% -0.02%
==========================================
Files 1374 1374
Lines 320382 320858 +476
Branches 10314 10387 +73
==========================================
+ Hits 308906 309320 +414
- Misses 9008 9057 +49
- Partials 2468 2481 +13 |
...ers/VisualBasic/Microsoft.NetCore.Analyzers/Runtime/BasicPreferDictionaryTryGetValueFixer.vb
Outdated
Show resolved
Hide resolved
is there still something that needs to be changed? |
@Youssef1313 or @mavasani could one of you do another review. Thanks |
@buyaa-n Can you please review? Thanks! |
@Miizukii for resolving merge conflicts |
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.
Thank you @Miizukii! LGTM
Changed Analyzer to first find ContainsKey() and than find all usages with the same unchanged key and dictionary reference in the guarded path.
Fixes: #6256, #6390, #6280