diff --git a/.github/workflows/_test.yml b/.github/workflows/_test.yml new file mode 100644 index 000000000..1d647a2bc --- /dev/null +++ b/.github/workflows/_test.yml @@ -0,0 +1,23 @@ +name: Unit Tests +on: workflow_call +jobs: + unit-tests: + runs-on: '${{ matrix.os }}' + strategy: + matrix: + os: + - ${{ vars.UBUNTU_VERSION }} + node-version: + - 12.x + - 14.x + - 16.x + steps: + - uses: actions/checkout@v2 + - name: 'Install node.js ${{ matrix.node-version }}' + uses: actions/setup-node@v2-beta + with: + node-version: '${{ matrix.node-version }}' + - name: Run unit tests + run: | + npm install + npm run ci diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml new file mode 100644 index 000000000..c9533fc8c --- /dev/null +++ b/.github/workflows/pull_request.yml @@ -0,0 +1,7 @@ +name: Continuous Integration +on: pull_request +jobs: + unit-tests: + # only run this job for forks + if: github.event.pull_request.head.repo.full_name != github.repository + uses: ./.github/workflows/_test.yml diff --git a/.github/workflows/push.yml b/.github/workflows/push.yml index 1b941bf99..6c8fc07f0 100644 --- a/.github/workflows/push.yml +++ b/.github/workflows/push.yml @@ -2,29 +2,11 @@ name: Continuous Integration on: push jobs: unit-tests: - runs-on: '${{ matrix.os }}' - strategy: - matrix: - os: - - ubuntu-20.04 - node-version: - - 12.x - - 14.x - - 16.x - steps: - - uses: actions/checkout@v2 - - name: 'Install node.js ${{ matrix.node-version }}' - uses: actions/setup-node@v2-beta - with: - node-version: '${{ matrix.node-version }}' - - name: Run unit tests - run: | - npm install - npm run ci + uses: ./.github/workflows/_test.yml npm-publish: needs: unit-tests - if: github.ref == 'refs/heads/master' && github.event_name == 'push' - runs-on: ubuntu-20.04 + if: github.ref == 'refs/heads/master' && needs.unit-tests.result == 'success' + runs-on: ${{ vars.UBUNTU_VERSION }} steps: - uses: actions/checkout@v2 - name: Install Node.js @@ -44,7 +26,7 @@ jobs: # note: github actions won't run a job if you don't call one of the status check functions, so `always()` is called since it evalutes to `true` if: ${{ always() && needs.unit-tests.result == 'success' && (needs.npm-publish.result == 'success' || needs.npm-publish.result == 'skipped') }} needs: [unit-tests, npm-publish] - runs-on: ubuntu-20.04 + runs-on: ${{ vars.UBUNTU_VERSION }} steps: - uses: actions/checkout@v2 - name: Build Docker images