-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change the prettier printWidth option back to the default, or something smaller #370
Comments
What is the default value? |
Default is 80, but it's not a hard limit. |
80 sounds way too small. |
This works well in most projects with a tabwidth of 2, but in speedometer with a tab width of 4, I believe 120 would work better indeed. |
I made a PR with a value of 120 so that we can see the result. Here is IMO a compelling example: 5c1e296#diff-2d8fb1419de7a1081886af05fca74332a3a6d307b322f87c0a127ab46a550344 I find the new format easier to read. But I could also be just used to it, so happy to hear other opinions. |
I would have picked a higher number like 160. |
The current value is
250
, this makes very long lines.Can we put it back to the default, or at least to a smaller value such a 120 if the default is too low because of the tab width?
The text was updated successfully, but these errors were encountered: