Merge pull request #1015 from ocsigen/remove-ppx_let-test-deps

Remove useless ppx_let-test-deps from Makefile
ocsigen · Apr 29, 2024 · 31703a1 · 31703a1
2 parents 9fbbf42 + f1bce01
commit 31703a1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -95,7 +95,7 @@ jobs:
 
       - run: opam install lwt_react lwt --deps-only --with-test
 
-      - run: opam install lwt_ppx --deps-only --with-test
+      - run: opam install lwt_ppx lwt_ppx_let --deps-only --with-test
         if: ${{ matrix.ppx == true }}
 
       - run: opam exec -- dune build --only-packages lwt_react,lwt
@@ -108,7 +108,7 @@ jobs:
       - run: opam exec -- dune runtest --only-packages lwt_ppx
         if: ${{ matrix.ppx == true }}
 
-      - run: opam exec -- make ppx_let-test-deps ppx_let-test
+      - run: opam exec -- make ppx_let-test
         if: ${{ matrix.ppx == true }}
 
   lint-opam:

diff --git a/Makefile b/Makefile
@@ -76,10 +76,6 @@ ppx_let-test :
 	dune build test/ppx_let/test.exe
 	dune exec test/ppx_let/test.exe
 
-.PHONY : ppx_let-test-deps
-ppx_let-test-deps :
-	opam install --yes --unset-root ppx_let
-
 .PHONY: clean
 clean :
 	dune clean