cd-slash
Formatter breaking code by adding new lines below `attr/3`
I’ve come across a few instances recently where the built-in formatter is breaking otherwise working code. As an example, take a Phoenix component with a moderately long default string attribute:
attr :text_color, :string, default: "text-gray-900 hover:text-primary-700 dark:text-gray-400 dark:hover:text-white"
attr :border, :string, default: "border focus:outline-none"
def button(assigns) do
end
This code works fine, except that with default settings the text_color attribute gets broken into 2 lines and a new line added below:
attr :text_color, :string,
default: "text-gray-900 hover:text-primary-700 dark:text-gray-400 dark:hover:text-white"
attr :border, :string, default: "border focus:outline-none"
def button(assigns) do
end
The additional line between the two attr/3 statements breaks the component and results in a compiler error.
I’ve worked around this by adding line_length: 120 to my .formatter.exs file, but this is not an ideal workaround. I’m struggling to find a way to override the rule that’s causing this to break; can it be set to ignore attr/3 line length, for example, or even not apply the rule in this file type?
Most Liked
LostKobrakai
Newlines shouldn’t be a problem. E.g. the CoreComponents phoenix generates do have such attr defintions as well and I’ve not seen anyone have problems with that. By my understanding attr simply aggregates all attr invokations between functions, no matter the whitespace or other (valid) syntax between the individual attr calls.
If that’s a problem on your end I’d argue that’s a bug somewhere, not a problem with the formatter.
Can you post the error you’re getting?
Last Post!
cd-slash
You’re completely right, I seem to have made an error elsewhere and somehow thought it was a compiler error. My mistake, thanks for your inputs!
Popular in Questions
Other popular topics
Latest Phoenix Threads
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #deployment
- #library
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #channels
- #elixirconf
- #exunit
- #discussion
- #code-sync
- #javascript
- #podcasts
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #elixir-ls
- #phoenix_html
- #iex
- #blog-post
- #graphql
- #genstage
- #ai
- #websockets
- #supervisor
- #elixirconf-us
- #advent-of-code
- #distillery
- #processes
- #forms
- #api
- #metaprogramming
- #hex
- #security









