fix: properly check prettier in CI

sourcegraph · Jan 11, 2019 · aa7f798 · aa7f798
1 parent f2f7744
commit aa7f798
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 2 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -10,7 +10,7 @@ jobs:
   include:
     - stage: test
       script:
-        - yarn run prettier
+        - yarn run prettier-check
         - yarn run tslint
         - yarn run build
     - stage: release

diff --git a/package.json b/package.json
@@ -30,6 +30,7 @@
     "tsconfig-schema": "download http://json.schemastore.org/tsconfig | json2ts | prettier --parser typescript > src/tsconfig.ts",
     "tslint-schema": "download http://json.schemastore.org/tslint | json2ts | prettier --parser typescript > src/tslint.ts",
     "prettier": "prettier '**/{*.{js?(on),ts?(x),scss},.*.js?(on)}' --write --list-different",
+    "prettier-check": "npm run prettier -- --write=false",
     "commitmsg": "commitlint -e $GIT_PARAMS",
     "semantic-release": "semantic-release"
   },

diff --git a/src/cli.ts b/src/cli.ts
@@ -238,6 +238,7 @@ async function main(): Promise<void> {
                 commitmsg: 'commitlint -e $GIT_PARAMS',
                 'semantic-release': 'semantic-release',
                 prettier: "prettier '**/{*.{js?(on),ts?(x),scss},.*.js?(on)}' --write --list-different",
+                'prettier-check': 'npm run prettier -- --write=false',
                 tslint: "tslint -c tslint.json -p tsconfig.json './src/*.ts?(x)' './*.ts?(x)'",
                 build: 'tsc -p .',
                 watch: 'tsc -p . -w',

diff --git a/src/travis.ts b/src/travis.ts
@@ -82,7 +82,7 @@ export async function initTravis({
                     {
                         stage: 'test',
                         script: [
-                            'yarn run prettier',
+                            'yarn run prettier-check',
                             'yarn run tslint',
                             'yarn run build',
                             ...(hasTests