Closed cphalen closed 1 year ago
lgtm chief ;)
Code changes look good! Can you update
CHANGELOG.md
with the changes you made? Maybe bump version to 2.0.0 and also put that version into the opam and dune files?
Oh great point I'll do this today thanks Will
Between OCaml 4.10.0 (previous max supported version) and OCaml 4.13.0 there were two major changes:
Stream
was moved out of the stdlib into an opam package calledcamlp-streams
, so I added the opam package to parts of the codebase that requiredStream
Parsetree
was updated which required some changes:Pwith_modtype
andPwith_modtypesubst
variants, these with constraints allow you to specify module type and module type substitutionsPsig_modtypesubst
variant which allows signatures to be defined with module type substitutionsPpat_construct
which was restructured slightly