Update linters

Author: sethvargo

.github/workflows/integration.yml

@@ -27,7 +27,7 @@ jobs:
 
       - uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
         with:
-          node-version: '20.x'
+          node-version-file: 'package.json'
 
       - name: 'npm build'
         run: 'npm ci && npm run build'
@@ -47,7 +47,15 @@ jobs:
             regional:${{ vars.REGIONAL_SECRET_NAME }}
 
       - name: 'outputs'
-        run: echo '${{ steps.secrets.outputs.token }}${{ steps.secrets.outputs.password }}${{ steps.secrets.outputs.regional }}'
+        env:
+          TOKEN: |-
+            ${{ steps.secrets.outputs.token || '' }}
+          PASSWORD: |-
+            ${{ steps.secrets.outputs.password || '' }}
+          REGIONAL: |-
+            ${{ steps.secrets.outputs.regional || '' }}
+        run: |-
+          echo '${TOKEN}${PASSWORD}${REGIONAL}'
 
       - id: 'secrets-encoded'
         name: 'secrets-encoded'
@@ -60,4 +68,12 @@ jobs:
             regional:${{ vars.REGIONAL_SECRET_NAME }}
 
       - name: 'outputs-encoded'
-        run: echo '${{ steps.secrets-encoded.outputs.token }}${{ steps.secrets-encoded.outputs.password }}${{ steps.secrets-encoded.outputs.regional }}'
+        env:
+          TOKEN: |-
+            ${{ steps.secrets-encoded.outputs.token || '' }}
+          PASSWORD: |-
+            ${{ steps.secrets-encoded.outputs.password || '' }}
+          REGIONAL: |-
+            ${{ steps.secrets-encoded.outputs.regional || '' }}
+        run: |-
+          echo '${TOKEN}${PASSWORD}${REGIONAL}'

.github/workflows/unit.yml

@@ -31,16 +31,10 @@ jobs:
 
       - uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
         with:
-          node-version: '20.x'
+          node-version-file: 'package.json'
 
       - name: 'npm build'
         run: 'npm ci && npm run build'
 
-      - name: 'npm lint'
-        # There's no need to run the linter for each operating system, since it
-        # will find the same thing 3x and clog up the PR review.
-        if: ${{ matrix.os == 'ubuntu-latest' }}
-        run: 'npm run lint'
-
       - name: 'npm test'
         run: 'npm run test'

bin/runTests.sh

@@ -16,4 +16,6 @@ set -eEuo pipefail
 FILES="$(node -e "process.stdout.write(require('node:fs').readdirSync('./', { recursive: true }).filter((e) => {return e.endsWith('.test.ts') && !e.startsWith('node_modules');}).sort().join(' '));")"
 
 set -x
+
+# shellcheck disable=SC2086
 exec node --require ts-node/register --test-reporter spec --test ${FILES}

package.json

@@ -10,6 +10,10 @@
     "format": "eslint . --fix",
     "test": "bash ./bin/runTests.sh"
   },
+  "engines": {
+    "node": "20.x",
+    "npm": "10.x"
+  },
   "repository": {
     "type": "git",
     "url": "https://github.com/google-github-actions/get-secretmanager-secrets"