Skip to content

Commit 6327685

Browse files
Trottdanielleadams
authored andcommitted
build: fail early in test-macos.yml
PR-URL: #41035 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
1 parent 6b6e1d0 commit 6327685

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

.github/workflows/test-macos.yml

+8
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,14 @@ jobs:
4343
python-version: ${{ env.PYTHON_VERSION }}
4444
- name: Environment Information
4545
run: npx envinfo
46+
# The `npm ci` for this step fails a lot as part of the Test step. Run it
47+
# now so that we don't have to wait 2 hours for the Build step to pass
48+
# first before that failure happens. (And if there's something about
49+
# `make run-ci -j2` that is causing the failure and the failure doesn't
50+
# happen anymore running this step here first, that's also useful
51+
# information.)
52+
- name: tools/doc/node_modules workaround
53+
run: make tools/doc/node_modules
4654
- name: Build
4755
run: make build-ci -j2 V=1 CONFIG_FLAGS="--error-on-warn"
4856
- name: Test

0 commit comments

Comments
 (0)