0.52 and trailing commas behavior on Kotlinlang formatting about ktfmt HOT 2 OPEN

Looks like this is intentional, going by the comments in the code

Personally, I would also prefer if anything with two arguments would be formatted on multiple lines with trailing commas.

Edit: after using 0.52 a bit, I feel strongly that more than one argument should go on separate lines. The whole point of trailing commas is to make merge conflicts easier, isn't it? That should not depend on whether it fits on one line.

from ktfmt.

I rolled back to even 0.50 and am seeing this constructor snafu showing up.

We had this:

class RealSettingsWorkflow @Inject constructor(
  private val oAuthTokenService: OAuthTokenService,
  @UserId private val userId: String,
  @VersionName private val versionName: String,
) : StatefulWorkflow<Unit, SettingsState, RootOutput, BodyAndOverlaysScreen<Screen, Overlay>>() {

is now:

class RealSettingsWorkflow
@Inject
constructor(
  private val oAuthTokenService: OAuthTokenService,
  @UserId private val userId: String,
  @VersionName private val versionName: String,
) :
  StatefulWorkflow<Unit, SettingsState, RootOutput, BodyAndOverlaysScreen<Screen, Overlay>>() {

I am using spotless, but I don't think it is causing this since it delegates to ktfmt. As an ex-Square, I do agree that the Square method is much easier to read. This current format it both ugly and makes me have to move my eyes down to see important information.

from ktfmt.