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

No space in by-name params #1079

Merged
merged 2 commits into from Nov 24, 2017

Conversation

3 participants
@vovapolu
Collaborator

vovapolu commented Nov 24, 2017

Fixing #1067

@olafurpg

Very nice! Just a comment on the name and then this is good to merge 👍

@olafurpg

LGTM 👍 will merge on green!

@olafurpg olafurpg merged commit 910a2ed into scalameta:master Nov 24, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@fommil

This comment has been minimized.

Show comment
Hide comment
@fommil

fommil Nov 24, 2017

omg, I want this yesterday!

fommil commented Nov 24, 2017

omg, I want this yesterday!

@olafurpg olafurpg added this to the v1.3.1 milestone Nov 24, 2017

@olafurpg

This comment has been minimized.

Show comment
Hide comment
@olafurpg

olafurpg Nov 25, 2017

Member

You can try this out today, the CI publishes artifacts on merge 😄

coursier fetch com.geirsson:scalafmt-core_2.12:1.3.0+14-910a2ed5 -r bintray:scalameta/maven
Member

olafurpg commented Nov 25, 2017

You can try this out today, the CI publishes artifacts on merge 😄

coursier fetch com.geirsson:scalafmt-core_2.12:1.3.0+14-910a2ed5 -r bintray:scalameta/maven
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment