Update default printer width to 80
from scalafmt docs about printer width:
- Keep in mind that 80 characters fit perfectly on a split laptop screen with regular resolution.
- GitHub mobile view only shows 80 characters and sometimes you might review code on your phone.
I don't have a strong opinion about width but split view is very important imo. Prettier 's default width is also 80 which is very popular in js community.
I'm also okay with making it configurable (80 / 100 / 120) without touching default width.
For what it's worth, the longer line width makes reviewing the transition from production Reason to Rescript a large pain. Transitioning would be much easier with an 80 char option, even if the default remains 120.
Any idea? I would happily work on this? /cc @IwanKaramazow @bobzhang
We still haven't decided what we're going to do here. Exploring a completely new printer architecture atm that operates independent of the line width.
Thank you for the answer! looking forward
any update on this? I would be happy to help
The rescript-lang/syntax repo is obsolete and will be archived soon. If this issue is still relevant, please reopen in the compiler repo (https://github.com/rescript-lang/rescript-compiler) or comment here to ask for it to be moved. Thank you for your contributions.