-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Enable 2 more Code Quality Rules #20814
Conversation
ec650bb
to
6340ab5
Compare
6340ab5
to
55048c8
Compare
The rules are fine, but |
Future PRs that make things public can choose to unseal them at the same time if it is appriopriate. |
55048c8
to
6c12ff9
Compare
6c12ff9
to
ddb5038
Compare
Seems like this needs a rebase again. |
ddb5038
to
52fdcc0
Compare
Rebased |
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.
OK
Following on from #20802, #20812
Enforces 2 code quality rules across the project. I have batched together a PR for these rules as the diff is large but boring.
See https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ for explanation of each rule.