Skip to content

Commit eafd6bb

Browse files
committed
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.
1 parent 0c09834 commit eafd6bb

File tree

2 files changed

+0
-26
lines changed

2 files changed

+0
-26
lines changed

.github/workflows/check-toc.yml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -35,19 +35,6 @@ jobs:
3535
- name: Checkout repository
3636
uses: actions/checkout@v2
3737

38-
- name: Get week number for use in cache key
39-
id: get-date
40-
run: |
41-
echo "::set-output name=week-number::$(date --utc '+%V')"
42-
43-
- name: Load dependencies cache
44-
uses: actions/cache@v2
45-
with:
46-
path: ~/.npm
47-
key: ${{ runner.os }}-node-markdown-toc-${{ steps.get-date.outputs.week-number }}
48-
restore-keys: |
49-
${{ runner.os }}-node-markdown-toc-
50-
5138
- name: Install markdown-toc
5239
run: sudo npm install --global markdown-toc
5340

.github/workflows/check-workflows.yml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -33,19 +33,6 @@ jobs:
3333
location: ${{ runner.temp }}/github-workflow-schema
3434
file-name: github-workflow.json
3535

36-
- name: Get week number for use in cache key
37-
id: get-date
38-
run: |
39-
echo "::set-output name=week-number::$(date --utc '+%V')"
40-
41-
- name: Load dependencies cache
42-
uses: actions/cache@v2
43-
with:
44-
path: ~/.npm
45-
key: ${{ runner.os }}-node-ajv-cli-${{ steps.get-date.outputs.week-number }}
46-
restore-keys: |
47-
${{ runner.os }}-node-ajv-cli-
48-
4936
- name: Install JSON schema validator
5037
run: sudo npm install --global ajv-cli
5138

0 commit comments

Comments
 (0)