Skip to content

Commit eafb9c8

Browse files
committed
CI: Don't run Mdx and coverage on older versions
Mdx isn't co-installable with Odoc on older compilers. Coverage dependencies are unused on these builds.
1 parent 25e53b8 commit eafb9c8

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

.github/workflows/build.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,12 @@ jobs:
2020
- 4.12.x
2121
- 4.14.x
2222
include:
23-
- os: ubuntu-latest # Enable coverage only on a single build
23+
- os: ubuntu-latest
2424
ocaml-compiler: 4.14.x
25+
# We don't need to compute coverage for more than one build
2526
send-coverage: true
27+
# Mdx tests Mdx tests
28+
run-mdx: true
2629
fail-fast: false
2730

2831
runs-on: ${{ matrix.os }}
@@ -44,21 +47,19 @@ jobs:
4447
- name: Install dependencies
4548
run: opam install -y --deps-only -t ./odoc.opam ./odoc-parser.opam
4649

47-
- name: Install bisect_ppx
48-
run: opam install bisect_ppx
49-
5050
- name: dune runtest
5151
run: opam exec -- dune runtest
5252

53-
# Run Mdx tests that are disabled by default.
5453
- name: Mdx tests
54+
if: matrix.run-mdx == true
5555
run: |
5656
opam install -y mdx
5757
opam exec -- dune build @runmdx
5858
5959
- name: Send coverage stats to Coveralls
6060
if: matrix.send-coverage == true
6161
run: |
62+
opam install bisect_ppx
6263
opam exec -- dune runtest --instrument-with bisect_ppx --force
6364
opam exec -- bisect-ppx-report send-to Coveralls
6465
env:

0 commit comments

Comments
 (0)