From 4e661aa6b5fe76aa892fb34828bdbd97f53b0f8e Mon Sep 17 00:00:00 2001 From: Riccardo Date: Fri, 12 Mar 2021 15:32:24 +0100 Subject: [PATCH] Added cache for vscode used for e2e tests (#498) * added caching of VS Code also for lint Even if linting doesn't require the vscode part of the cache, we are not separating the two cases so that we only have one cache to maintain, and linting being a faster task (and a task that should fail less than tests) will update the cache more often, speeding up the run of the tests afterwards --- .github/workflows/ci.yml | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 2b545449..ee66b736 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -20,13 +20,14 @@ jobs: uses: actions/setup-node@v1 with: node-version: '12' - - name: Restore Dependencies + - name: Restore Dependencies and VS Code test instance uses: actions/cache@v2 with: path: | node_modules */*/node_modules - key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }} + packages/foam-vscode/.vscode-test + key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }}-${{ hashFiles('packages/foam-vscode/src/test/run-tests.ts') }} - name: Install Dependencies run: yarn - name: Check Lint Rules @@ -47,13 +48,14 @@ jobs: uses: actions/setup-node@v1 with: node-version: '12' - - name: Restore Dependencies + - name: Restore Dependencies and VS Code test instance uses: actions/cache@v2 with: path: | node_modules */*/node_modules - key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }} + packages/foam-vscode/.vscode-test + key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }}-${{ hashFiles('packages/foam-vscode/src/test/run-tests.ts') }} - name: Install Dependencies run: yarn - name: Build Packages