Skip to content

Commit

Permalink
CI: Combine JS lint and JS check jobs
Browse files Browse the repository at this point in the history
Reduce the number of concurrent jobs.
This will require a branch protection rule update,
renaming `check-js` to Check JS` and removing `Lint`.
  • Loading branch information
adityasharad committed Jul 28, 2022
1 parent 6c23d76 commit 992d011
Showing 1 changed file with 6 additions and 11 deletions.
17 changes: 6 additions & 11 deletions .github/workflows/pr-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,17 +10,8 @@ on:
workflow_dispatch:

jobs:
lint-js:
name: Lint
runs-on: ubuntu-latest
timeout-minutes: 45

steps:
- uses: actions/checkout@v3
- name: Run Lint
run: npm run-script lint

check-js:
name: Check JS
runs-on: ubuntu-latest
timeout-minutes: 45

Expand All @@ -30,7 +21,11 @@ jobs:
node-types-version: [12.12, current]

steps:
- uses: actions/checkout@v3
- name: Checkout
uses: actions/checkout@v3

- name: Lint
run: npm run-script lint

- name: Update version of @types/node
if: matrix.node-types-version != 'current'
Expand Down

0 comments on commit 992d011

Please sign in to comment.