Merge pull request #176 from zazuko/ci-win-fix

Fix CI for Windows
zazuko · Apr 17, 2024 · e1b27ae · e1b27ae
2 parents a9cb254 + 42d6efa
commit e1b27ae
Show file tree

Hide file tree

Showing 3 changed files with 438 additions and 190 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -1,24 +1,29 @@
 name: Test validation
 
-on: [push, pull_request]
+on:
+  - push
+  - pull_request
 
 jobs:
   check-constraint:
-    runs-on: ${{ matrix.os }}
+    runs-on: ${{ matrix.env.os }}
     strategy:
       fail-fast: false
       matrix:
-        os:
-          - ubuntu-latest
-          - windows-latest
+        env:
+          - os: ubuntu-latest
+            node: 20
+          - os: windows-latest
+            node: 20.12.1 # Pinned, because of https://github.com/approvals/Approvals.NodeJS/issues/176
     steps:
       - name: Checkout
         uses: actions/checkout@v4
 
       - name: Set up Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: '20'
+          node-version: ${{ matrix.env.node }}
+          cache: npm
 
       - name: Install dependencies
         run: npm ci
@@ -40,13 +45,15 @@ jobs:
           path: test
 
   check-cube:
-    runs-on: ${{ matrix.os }}
+    runs-on: ${{ matrix.env.os }}
     strategy:
       fail-fast: false
       matrix:
-        os:
-          - ubuntu-latest
-          - windows-latest
+        env:
+          - os: ubuntu-latest
+            node: 20
+          - os: windows-latest
+            node: 20.12.1 # Pinned, because of https://github.com/approvals/Approvals.NodeJS/issues/176
         profile:
           - basic-cube-constraint
           - standalone-constraint-constraint
@@ -60,7 +67,8 @@ jobs:
       - name: Set up Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: '20'
+          node-version: ${{ matrix.env.node }}
+          cache: npm
 
       - name: Install dependencies
         run: npm ci