From ff1b5b6b144423f1d1c3592d1bf57d2bd27835fa Mon Sep 17 00:00:00 2001 From: Mathieu Barbin Date: Sun, 27 Oct 2024 18:36:14 +0100 Subject: [PATCH] Attempt to enable tests in more-ci --- .github/workflows/more-ci.yml | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/.github/workflows/more-ci.yml b/.github/workflows/more-ci.yml index 25d8fb1..ab97ea2 100644 --- a/.github/workflows/more-ci.yml +++ b/.github/workflows/more-ci.yml @@ -53,15 +53,10 @@ jobs: # janestreet-bleeding-external: https://github.com/janestreet/opam-repository.git#external-packages - name: Install dependencies - run: opam install ./provider.opam --deps-only --with-test + run: opam install ./provider.opam ./provider-tests.opam --deps-only --with-test - - name: Build - run: opam exec -- dune build @all -p provider - - # For now, we only run tests for the `provider` package. Effectively, this - # runs nothing because the tests are in the `provider-test` package. We - # configured this in preparation for running the tests for specific - # packages only in the future, once we have a dedicated test suite that - # can run on every matrix combination configured here. - - name: Run tests - run: opam exec -- dune build @runtest -p provider + # 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. + - name: Build & Run tests + run: opam exec -- dune build @all @runtest -p provider,provider-tests