Document recommended setup and improve docs overall#8510
Document recommended setup and improve docs overall#8510fisker merged 28 commits intoprettier:masterfrom
Conversation
| Now, format all files with Prettier: | ||
|
|
||
| ```bash | ||
| npx prettier@2.0.5 . --write |
There was a problem hiding this comment.
I added this part about npx when I found out that that's how some people run Prettier on CI without installing it as a dependency (no idea why they do that though). Would be good to keep this remark somewhere.
There was a problem hiding this comment.
“some people” – who?
Isn’t it better that we promote the “correct way” (adding it to "devDependencies" in package.json)?
Otherwise we’ll have to say something like “if you want to do it the wrong way, here’s a slightly less wrong way to do it”
There was a problem hiding this comment.
Makes sense. By "some people" I meant those who complained that Prettier broke their CI when 2.0 was released, but now I haven't been able to find those complaints. Probably it's indeed not that important.
There was a problem hiding this comment.
Nicely put. The tabs look good too. Might look even better without the top border.
Conflicts: package.json yarn.lock
Co-authored-by: Jed Fox <[email protected]>
This comment has been minimized.
This comment has been minimized.
|
Alright, all changes applied! |
Conflicts: website/.eslintrc.yml


Fixes #7828. Fixes #8464
✨Check it out✨