Skip to content

Commit

Permalink
chore(ci): Drop matrix for lint job (#2477)
Browse files Browse the repository at this point in the history
Fixes #2283

In order to avoid unnecessary duplication of effort in CI and to keep
the required Lint job name consistent as we upgrade the underlying
Node.js version, this change drops the test matrix and hard-codes the
platform and node version.
  • Loading branch information
kriskowal authored Oct 1, 2024
2 parents a74fcf3 + 49e2f5e commit 164a5a5
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,9 @@ jobs:

# begin macro

runs-on: ${{ matrix.platform }}
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
node-version: [18.x, 20.x]
platform: [ubuntu-latest]

steps:
- name: Checkout
Expand All @@ -33,10 +30,10 @@ jobs:
# without this, setup-node errors on mismatched yarn versions
- run: corepack enable

- name: Use Node.js ${{ matrix.node-version }}
- name: Use Node.js
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
node-version: 20.x
cache: yarn

- name: Install dependencies
Expand Down

0 comments on commit 164a5a5

Please sign in to comment.