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

Scalariform does not understand trailing comma #70

Open
marekzebrowski opened this issue Sep 8, 2017 · 6 comments
Open

Scalariform does not understand trailing comma #70

marekzebrowski opened this issue Sep 8, 2017 · 6 comments

Comments

@marekzebrowski
Copy link

reproducer

case class A(
a: Int,
b:Int,
)
@exp0nge
Copy link

exp0nge commented Jan 28, 2019

Is this ever going to be addressed?

@charpov
Copy link

charpov commented Apr 17, 2019

It's a small thing, but so annoying.

@exp0nge
Copy link

exp0nge commented Apr 17, 2019

@charpov use scalafmt instead

@aij
Copy link

aij commented Apr 17, 2019

@exp0nge scalafmt also has trouble with some trailing commas, though it's ok with most of them.

@charpov
Copy link

charpov commented Apr 17, 2019

I tried scalafmt once and didn't like it. I don't remember why now. I'll give it another shot.

@hubertta
Copy link

This is an issue with scalariform, not with sbt-scalariform. I made a pull request in scalariform that takes care of this problem: scala-ide/scalariform#296. Let's see if it gets merged and makes into a new release, then we could bump the scalariform version here in sbt-scalariform to make the fix available for this plugin.

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

No branches or pull requests

5 participants