Merge pull request #4194 from mermaid-js/chore/skip-setup-node-cachin…

…g-in-e2e-ci CI(e2e): Skip caching in `actions/setup-node`, as `cypress-io/github-action` already caches for us
mermaid-js · Mar 7, 2023 · 7bea44e · 7bea44e
2 parents 6b5221e + f0a7369
commit 7bea44e
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml
@@ -21,11 +21,7 @@ jobs:
 
       - name: Setup Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3
-        # Need to skip setup if Cypress run is skipped, otherwise an error
-        # is thrown since the pnpm cache step fails
-        if: ${{ ( env.CYPRESS_RECORD_KEY != '' ) || ( matrix.containers == 1 ) }}
         with:
-          cache: pnpm
           node-version: ${{ matrix.node-version }}
 
       # Install NPM dependencies, cache them correctly