search

ocaml-ppx / ppx_deriving_yojson

A Yojson codec generator for OCaml.
MIT License
155 stars 46 forks source link

Add missing dependency on result #156

Closed NathanReb closed 5 months ago

NathanReb commented 5 months ago

ppx_deriving_yojson depends on the result compatibility package but wasn't declaring this dependency explicitly.

This used to work because ppx_deriving also depended on result but since it will not be the case anymore starting from ppx_deriving.6.0.0, this needs to be updated.

It's likely that we will drop this dependency for the same reasons ppx_deriving did but in the meantime it does not cost much to update our opam file.

NathanReb commented 5 months ago

Ok nevermind, this has already been fixed and the result dependency stripped. Closing.