-
Notifications
You must be signed in to change notification settings - Fork 147
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
Add flag to skip insertion of spaces within multi imports. #69
Conversation
…f `import a.b.c.{d, e, f}` into `import a.b.c.{ d, e, f }`. Especially for Java experienced coders this looks more natural.
…ements individually (IntelliJ compatibility).
Matt Russell » scalariform #10 FAILURE |
Matt Russell » scalariform #11 SUCCESS |
Matt Russell » scalariform #12 SUCCESS |
Matt Russell » scalariform #13 SUCCESS |
Matt Russell » scalariform #14 SUCCESS |
Are there any plans to merge this pull request? In particular I'm interested in using the noSpacesAroundMultiImports stuff. If it the patches need to be cleaned up to merge, and @gbasler isn't around, I could take a crack at it. |
I'm around but I can't merge the pull request myself 😄 it's also quite a while since I've looked at this, so I wouln't mind if an extra pair of eyes could look at it 😃 |
I would love to see this feature merged as well |
+1 for noSpacesAroundMultiImports |
++1 Anyone can merge it? Is this community not activity anymore? |
The patch won't merge cleanly anymore, it would be a good first step to rebase on master. Unfortunately, @mdr isn't very active anymore. I'd love this patch as well. |
superseded by #127 |
Fix for #38