Skip to content
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

Usings sort #977

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Usings sort #977

wants to merge 11 commits into from

Conversation

BugAgain
Copy link
Contributor

Fixes CC0127: Sort Usings Directives by length #971

This is an experimental PR so you can give it a try if you want before you decide if this will be helpful to you.

P.S. I'm not that good at writing analyzers, I just wrote this because I wanted to use in a project that I'm working on. So, you can help me improve the code if you agree with this type of analyzer.

I wonder what you think.

Thx.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 91.631% when pulling 7e2f62c on BugAgain:usings_sort into 7b3e176 on code-cracker:master.

@giggio giggio self-assigned this May 22, 2018
@giggio giggio self-requested a review May 22, 2018 13:50
@giggio
Copy link
Member

giggio commented May 22, 2018

As mentioned on #971 let's wait to see if this proposal gathers more interest from the community.

@@ -154,4 +154,7 @@ Global
GlobalSection(ExtensibilityGlobals) = postSolution
SolutionGuid = {EF50C6AB-1421-41AE-8CB3-927AB24A69EA}
EndGlobalSection
GlobalSection(ExtensibilityGlobals) = postSolution
SolutionGuid = {15B937E4-F880-478B-B387-3438FDD895F7}
EndGlobalSection
Copy link
Contributor

@sharwell sharwell Sep 9, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 The changes in this file appear to be a mistake (possible merge conflict resolution?). It introduces an unexpected duplicate section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants