Skip to content

Commit

Permalink
feat(printWidth): set to 120 columns (#2)
Browse files Browse the repository at this point in the history
I'm open to debate on this. I'm not super hard sold on this or anything, just thought it'd be good to bring up and get everyone's thoughts on.
  • Loading branch information
Alex Van Camp authored Sep 19, 2019
1 parent b870319 commit cac41b6
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion prettier.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,18 @@ module.exports = {
/* A single quote requires pressing only one key,
* but a double quote requires holding shift.
* So, we find single quotes easier to type and prefer them. */
singleQuote: true
singleQuote: true,

/* It's true that Prettier warns pretty heavily against setting
* this higher than 80 columns: https://prettier.io/docs/en/options.html#print-width
*
* However, Prettier itself already disregards this warning in
* the name of readability: https://prettier.io/docs/en/rationale.html#imports
*
* Our friend Faulty has been using 120 for some time, and has had no issues.
* Indeed, all of Discord seems to use this setting.
*
* Lange: having tested this on some of my own real-world code,
* I agree with Faulty that it does make things easier to read.
printWidth: 120,
};

0 comments on commit cac41b6

Please sign in to comment.