Keep the breaking in new lines

[joan38]

Hi,

Original:

execute(request)
     .map(success => Xor.Right(success))
     .recover { case failure: Throwable => Xor.Left(failure) }

Scalafmt:

execute(request).map(success => Xor.Right(success)).recover { case failure: Throwable => Xor.Left(failure)

I would like Scalafmt to keep the breaking in new lines.

Using:

• 0.2.10
• --maxColumn 120

Thanks!

[olafurpg]

I have also experienced that scalafmt forces a select chain on a single line when I would like it to be multiline. It would be possible to skip the single line layout if there is a newline before the first select, which is .map(...) in this case. Scalafmt already does similar things for bodies of defs.

[olafurpg]

I think this can be fixed here: https://github.com/olafurpg/scalafmt/blob/master/core/src/main/scala/org/scalafmt/internal/Router.scala#L828

It should be something like Split(NoSplit, 0, ignoreIf = newlines > 0). However, I would like to keep the default behavior the way it is, so you need to guard against some setting like style.keepSelectChainLineBreaks. The setting doesn't exist yet so that needs to be added to ScalafmtStyle and Cli.

[melrief]

Pushed a PR that solves this.

[olafurpg]

Closed in #440

[joan38]

I don't quiet understand how to use this.
I tried:

keepSelectChainLineBreaks = true

and

breakChainOnFirstMethodDot = true

[olafurpg]

@joan38 https://github.com/olafurpg/scalafmt/pull/440#issuecomment-249995059

[olafurpg]

There's a section in the changelog describing more about the flag

[joan38]

Thanks @olafurpg for the hint!