Skip to content

Commit de819ed

Browse files
johnhaley81claude
andcommitted
Fix PPX test configuration: revert hardcoded paths to use opam binaries
- Change ppx/test/dune to use %{bin:reason-react-ppx} instead of hardcoded path - Change ppx/test/ppx.sh to use reason-react-ppx binary from PATH - PPX tests now work correctly with opam exec environment - Removes dependency on specific build directory structure 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 7942e3e commit de819ed

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

ppx/test/dune

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
(package reason-react-ppx)
33
(deps
44
(package reason-react)
5-
/home/me/external-repos/reason-react/_build/default/ppx/standalone.exe
5+
%{bin:reason-react-ppx}
66
%{bin:refmt}
77
%{bin:dune}
88
%{bin:jq}

ppx/test/ppx.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ if [ -z "$3" ]; then
1313
fi
1414

1515
refmt --parse re --print ml "$3" > output.ml
16-
/home/me/external-repos/reason-react/_build/default/ppx/standalone.exe --impl output.ml -o temp.ml
16+
reason-react-ppx --impl output.ml -o temp.ml
1717

1818
if [ "$2" == "ml" ]; then
1919
cat temp.ml

0 commit comments

Comments
 (0)