improve maintainability of github action

raphael-proust · raphael-proust · commit 8dfa0304324b · 2025-08-21T14:23:02.000+02:00
central place to set ppx-related values
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -43,31 +43,50 @@ jobs:
     runs-on: ${{ matrix.os }}
 
     steps:
+      - name: set ppx-related variables
+        id: config-ppx
+        run: |
+          case ${{ matrix.ocaml-compiler }} in
+            "4.08"|"4.09"|"4.10"|"4.11"|"4.12"|"4.13"|"4.14"|"5.0")
+              echo "ppx=false" >> "$GITHUB_OUTPUT"
+              echo "local-packages=lwt.opam,lwt_react.opam,lwt_retry.opam" >> "$GITHUB_OUTPUT"
+              ;;
+            "5.1"|"5.2"|"5.3")
+              echo "ppx=true" >> "$GITHUB_OUTPUT"
+              echo "local-packages=lwt.opam,lwt_ppx.opam,lwt_react.opam,lwt_retry.opam" >> "$GITHUB_OUTPUT"
+              ;;
+            *)
+              printf "unrecognised version %s\n" "${{ matrix.ocaml-compiler }}";
+              exit 1
+              ;;
+          esac
+
       - name: Checkout tree
         uses: actions/checkout@v5
 
       - name: Set-up OCaml
         uses: ocaml/setup-ocaml@v3
         with:
           ocaml-compiler: ${{ matrix.ocaml-compiler }}
+          opam-local-packages: ${{ steps.config-ppx.outputs.local-packages }}
 
       - run: opam install conf-libev
         if: ${{ matrix.libev == true }}
 
       - run: opam install ./lwt.opam ./lwt_react.opam ./lwt_retry.opam --deps-only --with-test
 
       - run: opam install ./lwt_ppx.opam --deps-only --with-test
-        if: ${{ contains(fromJSON('["5.1","5.2","5.3"]'), matrix.ocaml-compiler) }}
+        if: ${{ steps.config-ppx.outputs.ppx }}
 
-      - run: opam exec -- dune build -p lwt,lwt_react,lwt_retry
+      - run: opam exec -- dune build --only-packages lwt,lwt_react,lwt_retry
 
-      - run: opam exec -- dune build -p lwt_ppx
-        if: ${{ contains(fromJSON('["5.1","5.2","5.3"]'), matrix.ocaml-compiler) }}
+      - run: opam exec -- dune build --only-packages lwt_ppx
+        if: ${{ steps.config-ppx.outputs.ppx }}
 
-      - run: opam exec -- dune runtest -p lwt,lwt_react,lwt_retry
+      - run: opam exec -- dune runtest --only-packages lwt,lwt_react,lwt_retry
 
-      - run: opam exec -- dune runtest -p lwt_ppx
-        if: ${{ contains(fromJSON('["5.1","5.2","5.3"]'), matrix.ocaml-compiler) }}
+      - run: opam exec -- dune runtest --only-packages lwt_ppx
+        if: ${{ steps.config-ppx.outputs.ppx }}
 
   lint-opam:
     runs-on: ubuntu-latest
