Skip to content

Commit

Permalink
CI: Don't run Mdx and coverage on older versions
Browse files Browse the repository at this point in the history
Mdx isn't co-installable with Odoc on older compilers. Coverage
dependencies are unused on these builds.
  • Loading branch information
Julow committed Dec 12, 2023
1 parent 8ce3bba commit 7c035e6
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,12 @@ jobs:
- 4.04.x
- 4.14.x
include:
- os: ubuntu-latest # Enable coverage only on a single build
- os: ubuntu-latest
ocaml-compiler: 4.14.x
# We don't need to compute coverage for more than one build
send-coverage: true
# Mdx tests Mdx tests
run-mdx: true
fail-fast: false

runs-on: ${{ matrix.os }}
Expand All @@ -42,13 +45,15 @@ jobs:
run: opam install -y --deps-only -t ./odoc.opam ./odoc-parser.opam

- name: Install bisect_ppx
if: matrix.send-coverage == true
run: opam install bisect_ppx

- name: dune runtest
run: opam exec -- dune runtest

# Run Mdx tests that are disabled by default.
- name: Mdx tests
if: matrix.run-mdx == true
run: |
opam install -y mdx
opam exec -- dune build @runmdx
Expand Down

0 comments on commit 7c035e6

Please sign in to comment.