ci: move check-node-version to separate step

silvermine · May 3, 2024 · 2adf256 · 2adf256
1 parent 3ed6fb7
commit 2adf256
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -16,6 +16,7 @@ jobs:
                node-version-file: '.nvmrc'
          - run: npm i -g npm@10.5.0
          - run: npm ci
+         - run: npm run check-node-version
          - run: npm run standards
    test:
       needs: [ build ]
@@ -31,6 +32,5 @@ jobs:
             uses: actions/setup-node@v4
             with:
                node-version: ${{ matrix.node-version }}
-         - run: npm i -g npm@10.5.0
-         - run: npm ci # Reinstall the dependencies to ensure they install with the current version of node
+         - run: npm ci # Reinstall the dependencies to ensure they install with the node's version of npm
          - run: npm test
diff --git a/package.json b/package.json
@@ -4,7 +4,8 @@
   "description": "JS Code Standards for all SilverMine projects - eslint enforcement",
   "main": "index.js",
   "scripts": {
-    "test": "check-node-version --npm 10.5.0",
+    "check-node-version": "check-node-version --npm 10.5.0",
+    "test": "eslint",
     "release": "standard-version",
     "commitlint": "commitlint --from ee3b407",
     "eslint": "eslint .",