You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think I would ideally expect the formatting to happen exactly as mix format does, which would mean this:
<%=foo("bar") %>
Ideally it would also obey formatting configuration in .formatter.exs.
Perhaps it's not such a good idea for templates though. There are probably a lot of template helpers that feel cleaner without the parens. So the current behaviour doesn't hugely offend me, but I thought I'd open an issue for discussion about what the desired behaviour should be here.
The text was updated successfully, but these errors were encountered:
jonleighton
changed the title
Parentheses are not added
Parentheses are not added to function calls
May 11, 2021
This function call doesn't get formatted:
I think I would ideally expect the formatting to happen exactly as
mix format
does, which would mean this:Ideally it would also obey formatting configuration in
.formatter.exs
.Perhaps it's not such a good idea for templates though. There are probably a lot of template helpers that feel cleaner without the parens. So the current behaviour doesn't hugely offend me, but I thought I'd open an issue for discussion about what the desired behaviour should be here.
The text was updated successfully, but these errors were encountered: