Skip to content

Commit

Permalink
Merge PR coq#19958: CI: validate jobs also check stdlib
Browse files Browse the repository at this point in the history
Reviewed-by: ppedrot
Co-authored-by: ppedrot <[email protected]>
  • Loading branch information
coqbot-app[bot] and ppedrot authored Jan 6, 2025
2 parents ddfcf5c + b5c14e8 commit 253e9af
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -166,12 +166,13 @@ before_script:

# set "needs" when using
.validate-template:
stage: build-1
stage: build-2
interruptible: true
extends: .auto-use-tags
needs:
- not-a-real-job
script:
- for target in $CI_TARGETS; do dev/ci/ci-wrapper.sh "$target"; done
- cd _install_ci
- find lib/coq/ -name '*.vo' -fprint0 vofiles
- xargs -0 --arg-file=vofiles bin/coqchk -o -m -coqlib lib/coq/ > ../coqchk.log 2>&1 || touch coqchk.failed
Expand Down Expand Up @@ -579,10 +580,14 @@ test-suite:base+async:

validate:base:
extends: .validate-template
variables:
CI_TARGETS: "stdlib"
needs:
- build:base
- library:ci-stdlib
only: *full-ci

# we currently don't have a stdlib+32bit job
validate:base+32bit:
extends: .validate-template
needs:
Expand All @@ -594,8 +599,11 @@ validate:base+32bit:
validate:edge+flambda:
extends: .validate-template
image: $EDGE_IMAGE
variables:
CI_TARGETS: "stdlib"
needs:
- build:edge+flambda
- library:ci-stdlib+flambda
variables:
OPAM_VARIANT: "+flambda"
only: *full-ci
Expand Down

0 comments on commit 253e9af

Please sign in to comment.