Skip to content

Commit

Permalink
(optim/ci): don't build a second time for lint or test (#623)
Browse files Browse the repository at this point in the history
- yarn install already builds, so we can skip the build step of these
  scripts and run the post-build step of each instead
  • Loading branch information
agilgur5 authored Mar 17, 2020
1 parent 3a6e098 commit 6679c23
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/nodejs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
run: yarn install --frozen-lockfile

- name: Lint codebase
run: yarn lint
run: yarn lint:post-build

test:
runs-on: ${{ matrix.os }}
Expand All @@ -44,4 +44,4 @@ jobs:
run: yarn install --frozen-lockfile

- name: Test package
run: yarn test --runInBand
run: yarn test:post-build --runInBand

0 comments on commit 6679c23

Please sign in to comment.