segmentio / golines

A golang formatter that fixes long lines
MIT License
903 stars 56 forks source link

Do not move all parameters to individual lines #90

Open psankar opened 1 year ago

psankar commented 1 year ago

If a line exceeds 80 characters, we move all the parameters to each line on their own. Instead we could try to fit them in as lesser lines as possible. This would help avoid scrolling, improves readability. It is fine, even if this takes time.

In the screenshot below, the 3rd segment is more readable than the 2nd segment.

Screenshot from 2022-11-13 14-46-55

telemachus commented 1 year ago

In the screenshot below, the 3rd segment is more readable than the 2nd segment.

I know that this is bikeshedding, but I prefer the current defaults. I find it easier to read across or down rather than both across and down.

tonimelisma commented 9 months ago

@telemachus would you be open to adding this as a parameter please? Would a PR be approved?

gijswijs commented 5 months ago

Very much in favor of this being a parameter.