-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Add code example for CA2226 rule (#49086) #49087
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
Conversation
|
@kurnakovv Actually I think it's better to suppress CS0216 so this large project can continue to build without warnings in the future. |
Head branch was pushed to by a user without write access
|
@gewarren I'm not sure how to do it correctly, because this #pragma warning disable CS0216
// ...
#pragma warning restore CS0216doesn't work, and this dotnet_diagnostic.CS0216.severity = nonealso doesn't work I hope this is OK for code example #if false
// ...
#endif |
|
I'm trying to figure out why this rule even exists since it seems to be superseded by CS0216 for C# and BC33033 for Visual Basic.... |
|
Since this rule doesn't seem to fire and is a candidate for removal, let's not add a code example. |
Summary
Fixes #49086
Internal previews