Skip to content

Commit

Permalink
Avoid Node.js setup cache restore delays
Browse files Browse the repository at this point in the history
Downgrades to v3 to avoid a bug in GitHub actions/cache
actions/cache#810

Cache restore times for ~/.npm going over 5 minutes
  • Loading branch information
colinrotherham committed Dec 12, 2023
1 parent 082ab2b commit e811ef2
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/actions/setup-node/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ runs:

steps:
- name: Setup Node.js
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
id: setup-node

with:
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/sass.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
uses: actions/[email protected]

- name: Setup Node.js
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
with:
cache: npm
node-version: 8 # Node.js 8 supported by Dart Sass v1.0.0
Expand All @@ -47,7 +47,7 @@ jobs:
uses: actions/[email protected]

- name: Setup Node.js
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
with:
cache: npm
check-latest: true
Expand Down Expand Up @@ -77,7 +77,7 @@ jobs:
uses: actions/[email protected]

- name: Setup Node.js
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
with:
cache: npm
node-version: 4 # Node.js 4 supported by Node Sass v3.4.0
Expand All @@ -100,7 +100,7 @@ jobs:
uses: actions/[email protected]

- name: Setup Node.js
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
with:
cache: npm
check-latest: true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ jobs:
uses: ./.github/workflows/actions/build

- name: Change Node.js version
uses: actions/setup-node@v4.0.0
uses: actions/setup-node@v3.8.2
with:
node-version: ${{ matrix.node-version }}

Expand Down

0 comments on commit e811ef2

Please sign in to comment.