janestreet / ppx_yojson_conv

[@@deriving] plugin to generate Yojson conversion functions
MIT License
58 stars 8 forks source link

remove constraint on ppxlib #5

Closed giltho closed 2 years ago

giltho commented 3 years ago

Hi,

Surprisingly, it seems that ppx_yojson_conv is compatible with ppxlib >= 0.18 I've tested my fork by pinning it locally and it compiles and produces the right results (at least with ppx 0.20.0).

Would it be possible to remove the constraint and publish on opam :) ? If anything needs to be done before that, I can help.

cwong-ocaml commented 2 years ago

Apologies for the long response time; this got buried in my todo list. This no longer seems relevant, but please re-open if it is still an issue.

The official release of this package on opam doesn't seem to have an upper bound specified.