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
Ensure CI fails on non-prettier compliant PRs #693
Conversation
package.json
Outdated
"prettier --write --config ./.prettierrc --list-different \"common/**/*.ts\"", | ||
"prettier:diff-tsx": | ||
"prettier --write --config ./.prettierrc --list-different \"common/**/*.tsx\"", | ||
"prettier:diff": "npm run prettier:diff-ts && npm run prettier:diff-tsx", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need both prettier:diff-tsx
and prettier:diff-ts
, a single regex should be able to accommodate both.
.travis.yml
Outdated
@@ -20,6 +20,8 @@ install: | |||
|
|||
jobs: | |||
include: | |||
- stage: Prettier Diff |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why run it in a different stage?
.travis.yml
Outdated
@@ -20,6 +20,8 @@ install: | |||
|
|||
jobs: | |||
include: | |||
- stage: Prettier Diff | |||
script: npm run prettier:diff |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To confirm, --list-different
returns a non-zero exit code if there are any different files?
Closes MyEtherWallet/MyEtherWallet#601