From eafd6bbaf9671ce8c6d08dc89c582bec38cbb7a5 Mon Sep 17 00:00:00 2001 From: per1234 Date: Mon, 12 Apr 2021 02:45:29 -0700 Subject: [PATCH] Remove dependencies caching from workflows After I set up caching in the template workflows, doubts were raised about whether it provided any benefits. I don't know enough about this subject to make a call on that and I have been unable to get any more information on the subject. Since the caching significantly increases the complexity of the workflows, which may make them more difficult to maintain and contribute to, I think it's best to just remove all the caching for now. I hope to eventually be able to revisit this topic and restore caching in any workflows where it is definitely beneficial. --- .github/workflows/check-toc.yml | 13 ------------- .github/workflows/check-workflows.yml | 13 ------------- 2 files changed, 26 deletions(-) diff --git a/.github/workflows/check-toc.yml b/.github/workflows/check-toc.yml index 30d2765b..a841ac20 100644 --- a/.github/workflows/check-toc.yml +++ b/.github/workflows/check-toc.yml @@ -35,19 +35,6 @@ jobs: - name: Checkout repository uses: actions/checkout@v2 - - name: Get week number for use in cache key - id: get-date - run: | - echo "::set-output name=week-number::$(date --utc '+%V')" - - - name: Load dependencies cache - uses: actions/cache@v2 - with: - path: ~/.npm - key: ${{ runner.os }}-node-markdown-toc-${{ steps.get-date.outputs.week-number }} - restore-keys: | - ${{ runner.os }}-node-markdown-toc- - - name: Install markdown-toc run: sudo npm install --global markdown-toc diff --git a/.github/workflows/check-workflows.yml b/.github/workflows/check-workflows.yml index d6646c9b..b15b1d13 100644 --- a/.github/workflows/check-workflows.yml +++ b/.github/workflows/check-workflows.yml @@ -33,19 +33,6 @@ jobs: location: ${{ runner.temp }}/github-workflow-schema file-name: github-workflow.json - - name: Get week number for use in cache key - id: get-date - run: | - echo "::set-output name=week-number::$(date --utc '+%V')" - - - name: Load dependencies cache - uses: actions/cache@v2 - with: - path: ~/.npm - key: ${{ runner.os }}-node-ajv-cli-${{ steps.get-date.outputs.week-number }} - restore-keys: | - ${{ runner.os }}-node-ajv-cli- - - name: Install JSON schema validator run: sudo npm install --global ajv-cli