chore: CI: in quick mode, only Nix build runs the tests (#2998)

Following up on #2986, stop running the test suite in ci.yml in quick
mode; the test suite is run in the Nix job, and we do not need to run it
twice.

With a cold nix cache, when `lean` is rebuilt, not much changes, as both
jobs take ~20mins. But when `lean` is unchanged, the nix build should
be faster, and shaving off the (currently) 4mins in the CI.yaml run
should get us to a green PR sooner.

Another benefit is that we get the PR release sooner and even get it
when the test suite fails, which can be useful if you want to test
mathlib or other things before fixing the lean test suite.
This commit is contained in:
Joachim Breitner 2023-11-30 18:21:51 +01:00 committed by GitHub
parent 5b6e4faacd
commit f356d8830e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -371,10 +371,10 @@ jobs:
ulimit -c unlimited # coredumps
# exclude nonreproducible test
ctest -j4 --output-on-failure ${{ matrix.CTEST_OPTIONS }} < /dev/null
if: matrix.wasm || !matrix.cross
if: (matrix.wasm || !matrix.cross) && !needs.configure.outputs.quick
- name: Check Test Binary
run: ${{ matrix.binary-check }} tests/compiler/534.lean.out
if: ${{ !matrix.cross }}
if: ${{ !matrix.cross && !needs.configure.outputs.quick }}
- name: Build Stage 2
run: |
cd build