-
Notifications
You must be signed in to change notification settings - Fork 470
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
Suppress CA1860 when in expression tree #7123
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7123 +/- ##
==========================================
+ Coverage 96.43% 96.44% +0.01%
==========================================
Files 1412 1411 -1
Lines 336968 337282 +314
Branches 11143 11156 +13
==========================================
+ Hits 324946 325295 +349
+ Misses 9218 9183 -35
Partials 2804 2804 |
{ | ||
} | ||
} | ||
"""; |
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.
💡 The indentation here would be vastly improved if the closing """
was indented exactly 4 spaces relative to const
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.
Like this?
@mavasani When is it going to be released? Before .net 9? Thank you. |
@ArturDorochowicz You can reference this package version to get the fix. |
Fixes #7063