💚 Use alternate CI caching

Previously, stale local packages might be kept around due to caching symlinks within the node_modules directories. By caching yarns cache, we can avoid caching our own linked packages
percy · Sep 11, 2020 · dfd84a7 · dfd84a7
1 parent 33af5cb
commit dfd84a7
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 20 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -9,12 +9,13 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: 12
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
       - uses: actions/cache@v2
         with:
-          path: |
-            node_modules
-            packages/*/node_modules
-          key: ${{ runner.os }}/node-12/${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}/node-12/
+          path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
+          key: v1/${{ runner.os }}/node-12/${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v1/${{ runner.os }}/node-12/
       - run: yarn
       - run: yarn lint
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -9,13 +9,14 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: 10
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
       - uses: actions/cache@v2
         with:
-          path: |
-            node_modules
-            packages/*/node_modules
-          key: ${{ runner.os }}/node-10/${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}/node-10/
+          path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
+          key: v1/${{ runner.os }}/node-10/${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v1/${{ runner.os }}/node-10/
       - run: yarn
       - run: yarn build
       - uses: actions/upload-artifact@v2
@@ -48,13 +49,14 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: ${{ matrix.node }}
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
       - uses: actions/cache@v2
         with:
-          path: |
-            node_modules
-            packages/*/node_modules
-          key: ${{ runner.os }}/node-${{ matrix.node }}/${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}/node-${{ matrix.node }}/
+          path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
+          key: v1/${{ runner.os }}/node-${{ matrix.node }}/${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v1/${{ runner.os }}/node-${{ matrix.node }}/
       - uses: actions/download-artifact@v2
         with:
           name: dist

diff --git a/.github/workflows/typecheck.yml b/.github/workflows/typecheck.yml
@@ -9,12 +9,13 @@ jobs:
       - uses: actions/setup-node@v1
         with:
           node-version: 12
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "::set-output name=dir::$(yarn cache dir)"
       - uses: actions/cache@v2
         with:
-          path: |
-            node_modules
-            packages/*/node_modules
-          key: ${{ runner.os }}/node-12/${{ hashFiles('**/yarn.lock') }}
-          restore-keys: ${{ runner.os }}/node-12/
+          path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
+          key: v1/${{ runner.os }}/node-12/${{ hashFiles('**/yarn.lock') }}
+          restore-keys: v1/${{ runner.os }}/node-12/
       - run: yarn
       - run: yarn test:types