Skip to content
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

SA1648 throws an exception on delegates that use the <include> tag #3292

Merged
merged 5 commits into from Mar 10, 2021

Conversation

angelobreuer
Copy link
Contributor

Tracking issue: #3291

This PR includes the following changes:

@codecov
Copy link

codecov bot commented Jan 17, 2021

Codecov Report

Merging #3292 (e26b2f1) into master (23db6c0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3292   +/-   ##
=======================================
  Coverage   93.55%   93.55%           
=======================================
  Files        1017     1017           
  Lines      109990   110027   +37     
  Branches     3786     3786           
=======================================
+ Hits       102898   102935   +37     
  Misses       6141     6141           
  Partials      951      951           

Co-authored-by: Sam Harwell <sam@tunnelvisionlabs.com>
sharwell
sharwell previously approved these changes Jan 19, 2021
Copy link
Member

@sharwell sharwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementation looks good. I had a few suggestions to improve the tests.

Co-authored-by: Sam Harwell <sam@tunnelvisionlabs.com>
Had to make a new batch as some suggestions were not included

Co-authored-by: Sam Harwell <sam@tunnelvisionlabs.com>
@sharwell sharwell merged commit 03e0562 into DotNetAnalyzers:master Mar 10, 2021
@sharwell sharwell added this to the 1.2-beta.next milestone Mar 10, 2021
@sharwell
Copy link
Member

Thanks @angelobreuer !

@sharwell sharwell mentioned this pull request Mar 10, 2021
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.

SA1648 throws an exception on delegates that use the <include> tag
2 participants