tofu icon indicating copy to clipboard operation
tofu copied to clipboard

Consider changing scalafmt rules

Open catostrophe opened this issue 4 years ago • 3 comments

I do support the need for a forced formatter in a project with many contributors, but the current ruleset is awful and doesn't work properly in IDEA (has some wrong fields).

catostrophe avatar May 31 '20 06:05 catostrophe

@ReiReiRei attn

catostrophe avatar May 31 '20 09:05 catostrophe

@catostrophe do not mind. Rules had been set by @Odomontois so I have decided do not change them. Anyway, if rules are changed it meens a new large commit with reformated code. Anyway I think @catostrophe should create new issue for IDEA for better scalafmt support.

ReiReiRei avatar May 31 '20 12:05 ReiReiRei

I tried .scalafmt.conf from Izumi repo. Liked the result, but I would suggest a few changes:

  • maxColumn = 120
  • drop newlines.alwaysBeforeCurlyBraceLambdaParams = true

catostrophe avatar Aug 07 '20 14:08 catostrophe