Skip to content
This repository was archived by the owner on Feb 21, 2022. It is now read-only.

Fixed issue with multiple empty groupings causing whitespace. #25

Merged
merged 1 commit into from
Nov 26, 2016
Merged

Fixed issue with multiple empty groupings causing whitespace. #25

merged 1 commit into from
Nov 26, 2016

Conversation

matt-usurp
Copy link
Contributor

@matt-usurp matt-usurp commented Nov 12, 2016

As #24, some failing tests and an applied fix.

@matt-usurp matt-usurp changed the title Added failing test cases for additional whitespace when sorting use statements. Fixed issue with multiple empty groupings causing whitespace. Nov 25, 2016
@mmoreram
Copy link
Owner

@matt-usurp GREAT!

Can you please group all the commits in one?
After that, I will merge happily :D:D:D

Thanks!

@matt-usurp
Copy link
Contributor Author

Squashed, although can't you do that through the github screen these days?

Let me know if there is anything else.

@mmoreram mmoreram merged commit 20c16cd into mmoreram:master Nov 26, 2016
@mmoreram
Copy link
Owner

@matt-usurp awesome!

Big thanks :D

@matt-usurp matt-usurp deleted the issue-24-grouping-whitespace branch November 26, 2016 13:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants