Skip to content

Commit

Permalink
Move linting to the "check" job in CI
Browse files Browse the repository at this point in the history
It's not necessary to run `npm run check` in every compiler test job,
and the "check" job seems pretty useless as-is.
  • Loading branch information
CountBleck committed Oct 1, 2023
1 parent 9fd90aa commit 42d50cf
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/test.yml
Expand Up @@ -10,6 +10,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: dcodeIO/setup-node-nvm@master
with:
node-version: current
- name: "Check that distribution files are unmodified"
if: github.event_name == 'pull_request'
run: |
Expand All @@ -20,6 +23,10 @@ jobs:
else
printf "\nOK: Distributions files have not been modified.\n";
fi
- name: Install dependencies
run: npm ci --no-audit
- name: Check
run: npm run check
compiler:
name: "Compiler (${{ matrix.os }}, node ${{ matrix.node_version }})"
runs-on: ${{ matrix.os }}-latest
Expand All @@ -37,8 +44,6 @@ jobs:
run: npm ci --no-audit
- name: Build
run: npm run build
- name: Check
run: npm run check
- name: Test
run: npm test
bootstrap:
Expand Down

0 comments on commit 42d50cf

Please sign in to comment.