Skip to content
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

Prettify all JavaScript files #964

Merged
merged 4 commits into from
Sep 17, 2018
Merged

Prettify all JavaScript files #964

merged 4 commits into from
Sep 17, 2018

Conversation

yangshun
Copy link
Contributor

Motivation

Enable Prettier for v2 files. I took the opportunity to clean up all the prettier commands into one since we don't run them separately anyway. Also, use a **/*.js pattern to just run on every file. We have no use case for not running Prettier on any directory for now.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

CI.

Related PRs

NA.

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Sep 17, 2018
@yangshun yangshun requested a review from endiliey September 17, 2018 07:17
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Sep 17, 2018

Deploy preview for docusaurus-preview ready!

Built with commit ad840b9

https://deploy-preview-964--docusaurus-preview.netlify.com

Copy link
Contributor

@endiliey endiliey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might need to delete local prettier related files in v2 folder. We have another .prettierrc and prettier command in v2 folder

@yangshun
Copy link
Contributor Author

@endiliey Thanks for the reminder. I also removed the .prettierignore file in v2 as I think it's safe to remove.

@yangshun yangshun merged commit 9d4a5d5 into master Sep 17, 2018
@yangshun yangshun deleted the prettier branch September 17, 2018 07:35
@@ -5,5 +5,5 @@
"printWidth": 80,
"proseWrap": "never",
"singleQuote": true,
"trailingComma": "es5"
"trailingComma": "all"
Copy link
Contributor

@gustavohenke gustavohenke Nov 5, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This broke Node 6 support (LTS until April 2019) 🙃

But I assume my CI file could be tweaked to not build docs on all supported versions...
https://github.com/express-validator/express-validator/blob/b4f0c256d281fb917b59221f4bfa93c2d81cd91a/.travis.yml

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My apologies. We did mention that Docusaurus requires Node 8 and above: https://docusaurus.io/docs/en/installation#installing-docusaurus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants