Skip to content

Commit 1ceadcb

Browse files
committed
decouple check formatting job from build
1 parent 70e1815 commit 1ceadcb

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

.github/workflows/build.yml

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,8 @@ jobs:
9696
profiling_node:
9797
- 'packages/profiling-node/**'
9898
- 'dev-packages/e2e-tests/test-applications/node-profiling/**'
99+
any_code:
100+
- '!**/*.md'
99101
100102
101103
- name: Get PR labels
@@ -109,6 +111,8 @@ jobs:
109111
is_release: ${{ startsWith(github.ref, 'refs/heads/release/') }}
110112
changed_profiling_node: ${{ steps.changed.outputs.profiling_node == 'true' }}
111113
changed_ci: ${{ steps.changed.outputs.workflow == 'true' }}
114+
changed_any_code: ${{ steps.changed.outputs.any_code == 'true' }}
115+
112116
# When merging into master, or from master
113117
is_gitflow_sync: ${{ github.head_ref == 'master' || github.ref == 'refs/heads/master' }}
114118
has_gitflow_label:
@@ -123,6 +127,7 @@ jobs:
123127
runs-on: ubuntu-20.04
124128
timeout-minutes: 15
125129
if: |
130+
(needs.job_get_metadata.outputs.changed_any_code) &&
126131
(needs.job_get_metadata.outputs.is_gitflow_sync == 'false' && needs.job_get_metadata.outputs.has_gitflow_label == 'false')
127132
steps:
128133
- name: Check out base commit (${{ github.event.pull_request.base.sha }})
@@ -274,7 +279,7 @@ jobs:
274279

275280
job_check_format:
276281
name: Check file formatting
277-
needs: [job_get_metadata, job_build]
282+
needs: [job_get_metadata]
278283
timeout-minutes: 10
279284
runs-on: ubuntu-20.04
280285
steps:
@@ -286,12 +291,8 @@ jobs:
286291
uses: actions/setup-node@v4
287292
with:
288293
node-version-file: 'package.json'
289-
- name: Check dependency cache
290-
uses: actions/cache/restore@v4
291-
with:
292-
path: ${{ env.CACHED_DEPENDENCY_PATHS }}
293-
key: ${{ needs.job_build.outputs.dependency_cache_key }}
294-
fail-on-cache-miss: true
294+
- name: Install dependencies
295+
run: yarn install --frozen-lockfile
295296
- name: Check file formatting
296297
run: yarn lint:prettier && yarn lint:biome
297298

0 commit comments

Comments
 (0)