Skip to content

Commit

Permalink
Remove the unnecessary npm test in the build steps given that npm ci …
Browse files Browse the repository at this point in the history
…runs tests after the build
  • Loading branch information
Josh Kellendonk committed Dec 24, 2019
1 parent 77d7296 commit 3c409ab
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ jobs:
with:
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm test
env:
CI: true
- name: Coveralls
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ jobs:
with:
node-version: 12
- run: npm ci
- run: npm test

publish-npm:
needs: build
Expand All @@ -26,7 +25,6 @@ jobs:
node-version: 12
registry-url: https://registry.npmjs.org/
- run: npm ci
- run: npm test
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{secrets.npm_token}}
Expand Down

0 comments on commit 3c409ab

Please sign in to comment.