From c6fe3316ee149d46c4b60c581ce9fcedfb876c94 Mon Sep 17 00:00:00 2001 From: Mathieu Barbin Date: Mon, 16 Dec 2024 12:44:26 +0100 Subject: [PATCH] Disable implicit_transitive_deps construct in more-ci --- .github/workflows/more-ci.yml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/workflows/more-ci.yml b/.github/workflows/more-ci.yml index 5e8549b..5eaffd5 100644 --- a/.github/workflows/more-ci.yml +++ b/.github/workflows/more-ci.yml @@ -55,6 +55,14 @@ 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.