no-stack-dub-sack
commented
2 years ago @dschach while I admire the what this is trying to achieve, and I wish I used Prettier to begin with back then, this introduces a major set of changes that while I can be pretty sure are benign just does not seem like a high enough priority to consider this at the moment.
One concrete example I can point to is Prettier formatting certain bits of conditional logic and wrapping with parens which could change how that logic is executed. Our tests should catch this sort of thing but this just may not be required at the moment.
dschach
Lots of file formatting. Purely cosmetic, but brings all files in src folder to standard formatting, to make future development easier. (Better to deal with whitespace changes all at once now than piecemeal later, obscuring real changes.) The trailing commas in some of the objects are weird, but they're allowed in es5, so I'm not going to argue.