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.
Julow · Dec 12, 2023 · eafb9c8 · eafb9c8
1 parent 25e53b8
commit eafb9c8
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -20,9 +20,12 @@ jobs:
           - 4.12.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 }}
@@ -44,21 +47,19 @@ jobs:
       - name: Install dependencies
         run: opam install -y --deps-only -t ./odoc.opam ./odoc-parser.opam
 
-      - name: Install bisect_ppx
-        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
 
       - name: Send coverage stats to Coveralls
         if: matrix.send-coverage == true
         run: |
+          opam install bisect_ppx
           opam exec -- dune runtest --instrument-with bisect_ppx --force
           opam exec -- bisect-ppx-report send-to Coveralls
         env: