Also don't be surprised when fixes look cursed. The fix keeps all appended operands in their original place. No extra aligning and indentation.
This is necessary because
changing indentation messes up multi-line strings
case..of cases and let declarations are indentation sensitive, so adding for example a comma in front of the first line can lead to compiler errors
Replaces multiple
++in sequence with concat in a givenListSupplyStyle.For example, running
MultipleAppendToConcat.rule MultipleAppendToConcat.ApplyListonis fixed to
and for lists
fixed to
elm-reviewdoesn't have type inference, so for values likea ++ b ++ cthis rule doesn't provide a fix.Read the readme for why you would (not) want to enable this rule.
Also don't be surprised when fixes look cursed. The fix keeps all appended operands in their original place. No extra aligning and indentation. This is necessary because
case..ofcases andletdeclarations are indentation sensitive, so adding for example a comma in front of the first line can lead to compiler errorselm-formatwill safely prettify it anyway