search

mattjbray / ocaml-decoders

Elm-inspired decoders for Ocaml
https://mattjbray.github.io/ocaml-decoders/
Other
84 stars 8 forks source link

Updated compiler and fix Yojson deprecation warnings. #37

Closed idkjs closed 3 years ago

idkjs commented 3 years ago

Studying your code so did some touching up if you want it.

mattjbray commented 3 years ago

Probably about time we moved to Yojson.t. If you can just add a constraint on yojson to the version that added Yojson.Basic.t, etc. to the dune-project file here: https://github.com/mattjbray/ocaml-decoders/blob/e00bad1d2e4c2b1394aee9ae5a7a6fe3ab04ecec/dune-project#L117

Then run dune build to re-generate the opam files, I'll be happy to merge.