Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update SA1404 to not crash when attribute uses a namespace alias #3830

Merged

Conversation

bjornhellander
Copy link
Contributor

Fixes #3829

@bjornhellander bjornhellander force-pushed the feature/sa1404-global-3829 branch from 3fc7fed to 1f7ac37 Compare April 23, 2024 19:48
Copy link

codecov bot commented Apr 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.45%. Comparing base (045aba1) to head (1f7ac37).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3830   +/-   ##
=======================================
  Coverage   97.45%   97.45%           
=======================================
  Files         925      925           
  Lines      110040   110057   +17     
  Branches     3302     3303    +1     
=======================================
+ Hits       107236   107257   +21     
+ Misses       1839     1836    -3     
+ Partials      965      964    -1     

@bjornhellander
Copy link
Contributor Author

Ready for review

@sharwell sharwell merged commit 17b613d into DotNetAnalyzers:master Apr 29, 2024
19 checks passed
@sharwell sharwell added this to the 1.2-beta.next milestone Apr 29, 2024
@bjornhellander bjornhellander deleted the feature/sa1404-global-3829 branch April 29, 2024 14:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SA1404 throws NullReferenceException
2 participants