-
Notifications
You must be signed in to change notification settings - Fork 745
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
Apply analyzer (StyleCop) auto-fixes #5337
Conversation
I generated an API diff (which helped me track down places where |
40c4055
to
3989a84
Compare
Previous build: 25,628 warnings |
3989a84
to
021f81e
Compare
021f81e
to
72337e9
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.
Awesome
PR dnnsoftware#5337 implemented auto-fixes for style issues, but the renaming of backing fields sometimes caused a conflict which resulted in the property becoming recursive
PR #5337 implemented auto-fixes for style issues, but the renaming of backing fields sometimes caused a conflict which resulted in the property becoming recursive
Summary
I ran Code Cleanup from Visual Studio 2022 with the Fix analyzer warnings and errors set in EditorConfig option, and then resolved a couple of issues it created. I've closely reviewed and tweaked about half of the files.
It primarily adjusting naming conventions (removing underscores, making arguments and privates lowercase).
I also fixed an incorrect reference of SharpZipLib in Dnn.PersonaBar.Extensions (not sure why it wasn't causing trouble before).