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

Sorted imports #1071

Merged
merged 8 commits into from
Dec 26, 2016
Merged

Sorted imports #1071

merged 8 commits into from
Dec 26, 2016

Conversation

marcelofabri
Copy link
Collaborator

Continued from #991

Copy link
Collaborator

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

👍 thanks for finishing this!

@@ -38,6 +38,10 @@
[Marcelo Fabri](https://github.com/marcelofabri)
[#1066](https://github.com/realm/SwiftLint/issues/1066)

* Add a rule that enforces alphabetical sorting of imports.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Opt-in

@codecov-io
Copy link

codecov-io commented Dec 26, 2016

Current coverage is 83.29% (diff: 96.15%)

Merging #1071 into master will increase coverage by 0.04%

@@             master      #1071   diff @@
==========================================
  Files           147        148     +1   
  Lines          7189       7215    +26   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           5985       6010    +25   
- Misses         1204       1205     +1   
  Partials          0          0          

Powered by Codecov. Last update aecca7a...a73d917

@marcelofabri marcelofabri merged commit 570e627 into realm:master Dec 26, 2016
@marcelofabri marcelofabri deleted the sorted_imports branch December 26, 2016 20:28
@marcelofabri
Copy link
Collaborator Author

I squashed all commits because I thought that was @sberrevoets's idea, but it seems it removed his name from the history. I'm sorry 😰

@jpsim
Copy link
Collaborator

jpsim commented Dec 26, 2016

It's ok. His name will forever be credited in the changelog. The only place that matters :)

@sberrevoets
Copy link

Believe it or not, I was actually able to not open my laptop for a few days so didn't get a chance to look finish this up. Thanks for taking this over the finish line @marcelofabri!

@jpsim
Copy link
Collaborator

jpsim commented Dec 28, 2016

I applaud your self-restraint

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.

4 participants