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

smorimoto · web-flow · commit 31703a1fa893 · 2024-04-29T22:32:01.000+09:00
Remove useless ppx_let-test-deps from Makefile
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
