Skip to content

Commit

Permalink
Merge pull request #1916: Remove unnecessary CI jobs
Browse files Browse the repository at this point in the history
  • Loading branch information
victorlin authored Nov 21, 2024
2 parents f10cc77 + ff0e7db commit bf24816
Showing 1 changed file with 4 additions and 20 deletions.
24 changes: 4 additions & 20 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,6 @@ on:
name: ci

jobs:
build:
runs-on: ubuntu-latest
strategy:
matrix:
node: [16, 18, 20, 22]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
- run: node --version
# Build is implicit with the "prepare" life cycle script
- run: npm ci --loglevel verbose

unit-test:
runs-on: ubuntu-latest
strategy:
Expand All @@ -39,7 +25,7 @@ jobs:
- run: npm ci --loglevel verbose --ignore-scripts
- run: npm test

smoke-test:
build-and-smoke-test:
runs-on: ubuntu-latest
strategy:
matrix:
Expand All @@ -49,21 +35,19 @@ jobs:
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
# Build is implicit with the "prepare" life cycle script
- run: npm ci --loglevel verbose
- run: npm run get-data
- run: npx playwright install chromium
- run: npm run smoke-test

lint:
runs-on: ubuntu-latest
strategy:
matrix:
node: [16, 18, 20, 22]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
node-version: 18
# Note: --ignore-scripts skips an otherwise unnecessary Auspice build, but
# also skips life cycle scripts for dependencies. If the job fails here,
# try removing this flag.
Expand Down Expand Up @@ -115,7 +99,7 @@ jobs:
publish:
if: ${{ github.ref == 'refs/heads/release' }}
needs: [build, unit-test, smoke-test, lint, check-lockfile]
needs: [build-and-smoke-test, unit-test, lint, type-check, check-lockfile]
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand Down

0 comments on commit bf24816

Please sign in to comment.