Skip to content

Commit 6a6b07e

Browse files
committed
chore(ci): Clarify the testing scenario
Being in the middle, the "test all" case isn't as obvious nor the exceptions to it. This also ensures that the packages lower in the stack get tested before those higher in the stack, hopefully providing more actionable failures.
1 parent 61976b1 commit 6a6b07e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/main.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,8 @@ jobs:
151151
run: echo CARGO_CONTAINER_TESTS=1 >> $GITHUB_ENV
152152
if: matrix.os == 'ubuntu-latest'
153153

154-
- run: cargo test -p cargo
154+
# `benchsuite`, and some specific `cargo` cases, will be tested later
155+
- run: cargo test --exclude benchsuite
155156
- name: Clear intermediate test output
156157
run: ci/clean-test-output.sh
157158

@@ -170,7 +171,6 @@ jobs:
170171
run: 'cargo test -p cargo --test testsuite -- fix::'
171172
env:
172173
__CARGO_TEST_FORCE_ARGFILE: 1
173-
- run: cargo test --exclude cargo --exclude benchsuite
174174
- name: Check benchmarks
175175
run: |
176176
# This only tests one benchmark since it can take over 10 minutes to

0 commit comments

Comments
 (0)