-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuebugSomething isn't workingSomething isn't workingrepo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs
Description
Suggestion
This PR #7567 pinned PRIMARY_NODE_VERSION
variable in CI to >=20.6.1
typescript-eslint/.github/workflows/ci.yml
Lines 16 to 17 in e566a5d
env: | |
PRIMARY_NODE_VERSION: '>=20.6.1' |
But this variable is also compared to matrix.node-version
which van be 16
or 20
typescript-eslint/.github/workflows/ci.yml
Lines 194 to 217 in e566a5d
# collect coverage on the primary node version | |
# we don't collect coverage on other node versions so they run faster | |
# note that we don't do this as a single `run` with a flag because some | |
# packages don't collect coverage on purpose, so forcing `--coverage=true` | |
# would override the config | |
- name: Run unit tests with coverage for ${{ matrix.package }} | |
if: env.PRIMARY_NODE_VERSION == matrix.node-version | |
run: npx nx test ${{ matrix.package }} | |
env: | |
CI: true | |
- name: Run unit tests for ${{ matrix.package }} | |
if: env.PRIMARY_NODE_VERSION != matrix.node-version | |
run: npx nx test ${{ matrix.package }} --coverage=false | |
env: | |
CI: true | |
- name: Store coverage for uploading | |
if: env.PRIMARY_NODE_VERSION == matrix.node-version | |
uses: actions/upload-artifact@v3 | |
with: | |
name: ${{ matrix.package }}-coverage | |
path: packages/${{ matrix.package }}/coverage/lcov.info | |
# Sadly 1 day is the minimum | |
retention-days: 1 |
'>=20.6.1' == '20'
-> false
Thus, the coverage isn't collected and uploaded
I think we can unpin it since the current version of Node.js is already 20.10.0
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuebugSomething isn't workingSomething isn't workingrepo maintenancethings to do with maintenance of the repo, and not with code/docsthings to do with maintenance of the repo, and not with code/docs