Skip to content

Commit 1c9bb19

Browse files
committed
test: explain overcomplication of "test" GitHub Action in additional comments
1 parent c64aa7a commit 1c9bb19

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.github/workflows/test.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,12 @@ on:
1111
branches: [ "master" ]
1212

1313
env:
14+
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/
1415
NODE_LTE: 22.x
1516
NODE_LATEST: 25.x
1617

1718
jobs:
19+
# Copy node version numbers found in env into output variables, that can be used in strategy matrixes.
1820
nodeversions:
1921
runs-on: ubuntu-latest
2022
outputs:
@@ -27,11 +29,11 @@ jobs:
2729
echo "NODE_LATEST=${{ env.NODE_LATEST }}" >> $GITHUB_OUTPUT
2830
2931
nodejs:
32+
# Env variables are not available for strategy matrix, so we need to depend on output variables.
3033
needs: nodeversions
3134

3235
strategy:
3336
matrix:
34-
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/
3537
node-version: ["${{ needs.nodeversions.outputs.NODE_LTE }}", "${{ needs.nodeversions.outputs.NODE_LATEST }}"]
3638
os: [ubuntu-latest, windows-latest, macos-latest]
3739

0 commit comments

Comments
 (0)