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
Added more code style rules #19871
Added more code style rules #19871
Conversation
091cad9
to
31a0e0a
Compare
To resolve the issue of the style inside |
f0f5554
to
e757447
Compare
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.
LGTM. One fixup request.
Added two more commits - one to change |
Changed all currently present zero-length array allocations in the codebase to use `Array.Empty` instead.
Having this set to "none" disabled the IDE's option to add braces, whereas "silent" lets it do it on the user's request while still not suggesting it on its own.
3d14c3b
to
666e2e5
Compare
Rebased. Only changes are to the comments in |
To order them by their number.
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.
This is a continuation of my recent efforts to finally have our toolchain enforce our long-standing code style rules.
Follow-up of #19747 and #19801, this adds
twothree more rules to.editorconfig
to treattwothree more suggestions as warnings, which the CI then treats as errors.Calling @RoosterDragon.