Skip to content

Commit 0e818c1

Browse files
committed
ci: remove caching from release tests as it breaks with our removed package-lock
1 parent 40b6ebd commit 0e818c1

File tree

1 file changed

+11
-27
lines changed

1 file changed

+11
-27
lines changed

.github/workflows/tests-release.yml

+11-27
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ jobs:
2323
runs-on: ubuntu-latest
2424

2525
steps:
26-
- uses: actions/checkout@v2
27-
- uses: actions/setup-node@v2
26+
- uses: actions/checkout@v3
27+
- uses: actions/setup-node@v3
2828
with:
2929
node-version: '14'
3030
# install to create local package-lock.json but don't cache the files
@@ -43,21 +43,13 @@ jobs:
4343
node: [14, 16, 18]
4444
steps:
4545
- name: Checkout ${{ matrix.node }}
46-
uses: actions/checkout@v2
46+
uses: actions/checkout@v3
4747

4848
- name: Setup node ${{ matrix.node }}
49-
uses: actions/setup-node@v2
49+
uses: actions/setup-node@v3
5050
with:
5151
node-version: ${{ matrix.node }}
5252

53-
- name: Cache dependencies ${{ matrix.node }}
54-
uses: actions/cache@v1
55-
with:
56-
path: ~/.npm
57-
key: ${{ runner.os }}-node-${{ matrix.node }}-${{ hashFiles('**/package-lock.json') }}
58-
restore-keys: |
59-
${{ runner.os }}-node-${{ matrix.node }}
60-
6153
# for this workflow we also require npm audit to pass
6254
- run: npm i
6355
- run: npm run test:coverage
@@ -85,28 +77,20 @@ jobs:
8577
steps:
8678
# checkout this repo
8779
- name: Checkout ${{ matrix.node }}
88-
uses: actions/checkout@v2
80+
uses: actions/checkout@v3
8981

9082
# checkout express-adapter repo
9183
- name: Checkout express-adapter ${{ matrix.node }}
92-
uses: actions/checkout@v2
84+
uses: actions/checkout@v3
9385
with:
9486
repository: node-oauth/express-oauth-server
9587
path: github/testing/express
9688

9789
- name: Setup node ${{ matrix.node }}
98-
uses: actions/setup-node@v2
90+
uses: actions/setup-node@v3
9991
with:
10092
node-version: ${{ matrix.node }}
10193

102-
- name: Cache dependencies ${{ matrix.node }}
103-
uses: actions/cache@v1
104-
with:
105-
path: ~/.npm
106-
key: ${{ runner.os }}-node-${{ matrix.node }}-node-oauth/express-oauth-server-${{ hashFiles('github/testing/express/**/package-lock.json') }}
107-
restore-keys: |
108-
${{ runner.os }}-node-${{ matrix.node }}-node-oauth/express-oauth-server
109-
11094
# in order to test the adapter we need to use the current checkout
11195
# and install it as local dependency
11296
# we just cloned and install it as local dependency
@@ -122,8 +106,8 @@ jobs:
122106
runs-on: ubuntu-latest
123107
needs: [integrationtests]
124108
steps:
125-
- uses: actions/checkout@v2
126-
- uses: actions/setup-node@v2
109+
- uses: actions/checkout@v3
110+
- uses: actions/setup-node@v3
127111
with:
128112
node-version: 14
129113
registry-url: https://registry.npmjs.org/
@@ -139,8 +123,8 @@ jobs:
139123
contents: read
140124
packages: write
141125
steps:
142-
- uses: actions/checkout@v2
143-
- uses: actions/setup-node@v2
126+
- uses: actions/checkout@v3
127+
- uses: actions/setup-node@v3
144128
with:
145129
# we always publish targeting the lowest supported node version
146130
node-version: 14

0 commit comments

Comments
 (0)