cache node modules on win, correctly pass HAS_SECRET

.github/workflows/build-windows.yml

@@ -18,6 +18,19 @@ jobs:
           fetch-depth: 0
           # we need fetch-depth 0 so setuptools_scm can resolve tags
 
+      - name: Cache node modules
+      uses: actions/cache@v2
+      env:
+        cache-name: cache-node-modules
+      with:
+        # npm cache files are stored in `~/.npm` on Linux/macOS
+        path: ~/.npm
+        key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
+        restore-keys: |
+          ${{ runner.os }}-build-${{ env.cache-name }}-
+          ${{ runner.os }}-build-
+          ${{ runner.os }}-
+
       - name: Test for secrets access
         id: check_secrets
         shell: bash
@@ -43,7 +56,7 @@ jobs:
       - name: Build Windows installer with build_scripts\build_windows.ps1
         env:
           WIN_CODE_SIGN_PASS: ${{ secrets.WIN_CODE_SIGN_PASS }}
-          HAS_SECRET: steps.check_secrets.outputs.HAS_SECRET
+          HAS_SECRET: ${{ steps.check_secrets.outputs.HAS_SECRET }}
         run: |
           $env:path="C:\Program` Files` (x86)\Microsoft` Visual` Studio\2019\Enterprise\SDK\ScopeCppSDK\vc15\VC\bin\;$env:path"
           $env:path="C:\Program` Files` (x86)\Windows` Kits\10\App` Certification` Kit;$env:path"