Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Usings sort#977

Open
BugAgain wants to merge 11 commits intocode-cracker:mastercode-cracker/code-cracker:masterfrom
BugAgain:usings_sortBugAgain/code-cracker:usings_sortCopy head branch name to clipboard
Open

Usings sort#977
BugAgain wants to merge 11 commits intocode-cracker:mastercode-cracker/code-cracker:masterfrom
BugAgain:usings_sortBugAgain/code-cracker:usings_sortCopy head branch name to clipboard

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.

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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.