chore: improve ci speed by splitting lint and test (#1252)

Co-authored-by: Pascal Jufer <pascal-jufer@bluewin.ch>

.github/workflows/node.js.yml

@@ -5,34 +5,32 @@ name: Node.js CI
 on: [push]
 
 jobs:
-  build:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        node-version: [14, 16, 18]
-        os: [ubuntu-latest, macOS-latest]
+  lint:
+    runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm --version
-    - run: npm ci --ignore-scripts
-    - run: npm test
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+      - run: npm ci --ignore-scripts
+      - run: npm run lint
 
-  # Node 14 comes with npm 6 which has issues on Windows
-  # Skipping tests on Windows with Node 14
-  build-windows:
-    runs-on: ${{ matrix.os }}
+  test:
     strategy:
       matrix:
-        node-version: [16, 18]
-        os: [windows-latest]
+        node-version: [14, 16, 18]
+        os: [ubuntu-latest, macOS-latest, windows-latest]
+        # Node.js 14 comes with npm 6 which has issues on Windows
+        # therefore skipping tests on Windows with Node.js 14
+        exclude:
+          - os: windows-latest
+            node-version: 14
+    runs-on: ${{ matrix.os }}
     steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm --version
-    - run: npm ci --ignore-scripts
-    - run: npm test
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm --version
+      - run: npm ci --ignore-scripts
+      - run: npm test