From 2c7bc01cd8da2848d300a602019d4c53751122fb Mon Sep 17 00:00:00 2001 From: Henry Lee Date: Tue, 2 Jun 2026 08:15:12 +0000 Subject: [PATCH 1/2] Slim CI Format job to formatting tooling only The Format job ran `opam install . --deps-only`, pulling in the project's library dependencies. But `dune build @fmt` only runs ocamlformat and diffs; it never compiles the project, so those libraries are unused there. Replace the deps install with an explicit `dune` + `ocamlformat` install. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/ci.yml | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 81e0026..f185b73 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -51,11 +51,8 @@ jobs: ocaml-compiler: "5.4" dune-cache: true - - name: Install dependencies - run: opam install . --deps-only - - - name: Install formatter - run: opam install ocamlformat.0.29.0 + - name: Install dune and ocamlformat + run: opam install dune ocamlformat.0.29.0 - name: Check formatting run: opam exec -- dune build @fmt From 87dea3a807a3230ab5b3b4780112bb9f9013491b Mon Sep 17 00:00:00 2001 From: Henry Lee Date: Tue, 2 Jun 2026 08:15:42 +0000 Subject: [PATCH 2/2] Stop pinning the ocamlformat version Drop the `0.29.0` pin from both the CI formatter install and the `.ocamlformat` `version` field so formatting tracks the installed ocamlformat rather than a fixed release. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/ci.yml | 2 +- .ocamlformat | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index f185b73..0ca7989 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -52,7 +52,7 @@ jobs: dune-cache: true - name: Install dune and ocamlformat - run: opam install dune ocamlformat.0.29.0 + run: opam install dune ocamlformat - name: Check formatting run: opam exec -- dune build @fmt diff --git a/.ocamlformat b/.ocamlformat index 67b077f..0619e80 100644 --- a/.ocamlformat +++ b/.ocamlformat @@ -1,2 +1 @@ -version = 0.29.0 profile = janestreet