Skip to content

Commit 5fc9563

Browse files
authored
ci: Fix build (#514)
Update CI to turn off caching from setup-node when no install is run in the detect changesets job Also updated to node 22 in most places and removed node 18
1 parent 9372956 commit 5fc9563

File tree

6 files changed

+399
-204
lines changed

6 files changed

+399
-204
lines changed

.github/workflows/automate_dependabot_changeset.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,10 @@ jobs:
3434
git config --global user.name 'Github changeset workflow'
3535
3636
- name: Set up Node
37-
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
37+
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5
3838
with:
39-
node-version: 20
40-
39+
node-version: 22
40+
4141
- name: Fetch previous commit for check
4242
run: git fetch origin '${{ github.event.pull_request.base.sha }}'
4343

.github/workflows/ci.yml

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@ concurrency:
1111

1212
permissions:
1313
contents: read
14-
14+
pull-requests: write
15+
1516
jobs:
1617
find-changed-workspaces:
1718
name: Detect workspace changes
@@ -28,10 +29,11 @@ jobs:
2829
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v4
2930

3031
- name: Set up Node
31-
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
32+
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5
3233
with:
33-
node-version: 20
34+
node-version: 22
3435
registry-url: https://registry.npmjs.org/ # Needed for auth
36+
package-manager-cache: false
3537

3638
- name: Fetch previous commit for check
3739
run: git fetch origin '${{ github.event.pull_request.base.sha }}'
@@ -50,7 +52,7 @@ jobs:
5052
strategy:
5153
matrix:
5254
workspace: ${{ fromJSON(needs.find-changed-workspaces.outputs.workspaces) }}
53-
node-version: [18.x, 20.x]
55+
node-version: [20, 22]
5456
fail-fast: false
5557
defaults:
5658
run:
@@ -76,7 +78,7 @@ jobs:
7678
ref: ${{ github.event.workflow_run.head_branch }}
7779

7880
- name: Set up Node ${{ matrix.node-version }}
79-
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
81+
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5
8082
with:
8183
node-version: ${{ matrix.node-version }}
8284
registry-url: https://registry.npmjs.org/ # Needed for auth
@@ -141,9 +143,10 @@ jobs:
141143
with:
142144
ref: ${{ github.event.workflow_run.head_branch }}
143145
- name: Setup node
144-
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
146+
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5
145147
with:
146-
node-version: 20.x
148+
node-version: 22
149+
147150
- name: Install root dependencies
148151
run: yarn install --immutable
149152
- name: Verify lockfile duplicates
@@ -166,6 +169,8 @@ jobs:
166169
167170
dependency-review:
168171
runs-on: ubuntu-latest
172+
permissions:
173+
pull-requests: write
169174
steps:
170175
- name: Harden Runner
171176
uses: step-security/harden-runner@f4a75cfd619ee5ce8d5b864b0d183aff3c69b55a # v2.13.1
@@ -174,5 +179,9 @@ jobs:
174179

175180
- name: "Checkout Repository"
176181
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
182+
177183
- name: "Dependency Review"
178184
uses: actions/dependency-review-action@595b5aeba73380359d98a5e087f648dbb0edce1b # v4.7.3
185+
with:
186+
warn-only: true
187+
comment-summary-in-pr: on-failure

.github/workflows/release.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,12 @@ jobs:
2222
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v4
2323

2424
- name: Set up Node
25-
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
25+
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5
2626
with:
27-
node-version: 20
27+
node-version: 22
2828
registry-url: https://registry.npmjs.org/ # Needed for auth
29-
29+
package-manager-cache: false
30+
3031
- name: Fetch previous commit for check
3132
run: git fetch origin '${{ github.event.before }}'
3233

.github/workflows/release_workspace.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
- name: Set up Node
4949
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
5050
with:
51-
node-version: 20
51+
node-version: 22
5252
registry-url: https://registry.npmjs.org/ # Needed for auth
5353

5454
- name: Get yarn cache directory path
@@ -113,7 +113,7 @@ jobs:
113113
- name: Set up Node
114114
uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v4
115115
with:
116-
node-version: 20
116+
node-version: 22
117117
registry-url: https://registry.npmjs.org/ # Needed for auth
118118

119119
- name: Get yarn cache directory path

workspaces/confluence/package.json

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
},
88
"scripts": {
99
"tsc": "tsc",
10-
"tsc:full": "tsc --skipLibCheck false --incremental false",
10+
"tsc:full": "tsc --skipLibCheck true --incremental false",
1111
"build:all": "backstage-cli repo build --all",
1212
"build:api-reports": "yarn build:api-reports:only --tsc",
1313
"build:api-reports:only": "backstage-repo-tools api-reports -o ae-wrong-input-file-type --validate-release-tags",
@@ -45,11 +45,7 @@
4545
},
4646
"resolutions": {
4747
"@types/react": "^18",
48-
"@types/react-dom": "^18",
49-
"tough-cookie": "4.1.3",
50-
"@kubernetes/client-node": "^1.0.0-rc4",
51-
"@grpc/grpc-js": "1.10.9",
52-
"ws": "6.2.3"
48+
"@types/react-dom": "^18"
5349
},
5450
"prettier": "@spotify/prettier-config",
5551
"lint-staged": {

0 commit comments

Comments
 (0)