DO NOT MERGE

mbarbin · mbarbin · commit e20fdc3ba28b · 2025-05-23T09:58:44.000+02:00
- experiment with OCaml 4 and `(implicit_transitive_deps false)`.
diff --git a/.github/workflows/ocaml-4-ci.yml b/.github/workflows/ocaml-4-ci.yml
@@ -38,14 +38,6 @@ jobs:
       #     janestreet-bleeding: https://github.com/janestreet/opam-repository.git
       #     janestreet-bleeding-external: https://github.com/janestreet/opam-repository.git#external-packages
 
-      # This construct is not well supported by older OCaml versions. Given that
-      # we already check the build with this option in the main CI job, we
-      # disable it here unconditionally for simplicity.
-      - name: Edit dune-project
-        shell: pwsh
-        run: |
-          (Get-Content dune-project) -notmatch '\(implicit_transitive_deps false\)' | Set-Content dune-project
-
       # We build and run tests for a subset of packages. More tests are run in
       # the development workflow and as part of the main CI job. These are the
       # tests that are checked for every combination of os and ocaml-compiler.
