Should this code:
currentNamespace.class
(if showLabels then
[]
else
[ LabelText ]
)
automatically become:
currentNamespace.class <|
if showLabels then
[]
else
[ LabelText ]
"When possible" is important, especially considering things like https://github.com/avh4/elm-format/issues/362
I like this idea in general, but although this works...
something
|> currentNamespace.class
(if showLabels then
[]
else
[ LabelText ]
)
...it would be bad to rewrite that into a precedence conflict!
something
|> currentNamespace.class <|
if showLabels then
[]
else
[ LabelText ]
I like this idea overall. In the vein of "elm-format makes consistent style decisions for you," I think "avoid multiline expressions that are wrapped in parentheses" is a good one.
Most helpful comment
I like this idea overall. In the vein of "
elm-formatmakes consistent style decisions for you," I think "avoid multiline expressions that are wrapped in parentheses" is a good one.