Skip to content

Commit

Permalink
chore: exclude Node 14 and 16 on macos (#127)
Browse files Browse the repository at this point in the history
* chore: exclude Node 14 and 16 on macos

Related to fastify/.github#37.

This is just a proposal, probably not comprehensive of all the places where a similar change may have to occur, simply to capture that this is possible (and I didn't know until now) to exclude combinations, as documented in https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs#excluding-matrix-configurations

Signed-off-by: Simone Busoli <simone.busoli@gmail.com>

* remove os matrix from package

---------

Signed-off-by: Simone Busoli <simone.busoli@gmail.com>
Co-authored-by: Gürgün Dayıoğlu <hey@gurgun.day>
  • Loading branch information
simoneb and gurgunday authored Apr 29, 2024
1 parent a966a7e commit 04a2d6a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/plugins-ci-package-manager.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,13 @@ on:
jobs:
pnpm:
name: pnpm
runs-on: ${{ matrix.os }}
runs-on: ubuntu-latest
permissions:
contents: read
strategy:
matrix:
node-version: ${{ fromJson(inputs.node-versions) }}
os: [ubuntu-latest]
pnpm-version: [8]

steps:
- name: Check out repo
uses: actions/checkout@v4
Expand All @@ -45,13 +43,12 @@ jobs:

yarn:
name: Yarn
runs-on: ${{ matrix.os }}
runs-on: ubuntu-latest
permissions:
contents: read
strategy:
matrix:
node-version: ${{ fromJson(inputs.node-versions) }}
os: [ubuntu-latest]
steps:
- name: Check out repo
uses: actions/checkout@v4
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/plugins-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,11 @@ jobs:
matrix:
node-version: ${{ fromJson(inputs.node-versions) }}
os: [macos-latest, ubuntu-latest, windows-latest]
exclude:
- os: macos-latest
node-version: 14
- os: macos-latest
node-version: 16
steps:
- name: Check out repo
uses: actions/checkout@v4
Expand Down

0 comments on commit 04a2d6a

Please sign in to comment.