search

upenn-cis1xx / camelot

A fully-modular OCaml style linter
https://opam.ocaml.org/packages/camelot/
Apache License 2.0
44 stars 5 forks source link

Updating the OPAM #90

Open richard-zhang opened 1 year ago

richard-zhang commented 1 year ago

Could you do a release on the current version on the master?

I can see the current version available on OPAM is not updated hence not compatible with 4.14 above compiler.

ksromanov commented 1 year ago

Second the request. Please update OPAM version.

hanshoglund commented 1 year ago

IIUC code changes changes (not just release) are required for 4.14 support.

For example Pext_decl now takes 3 arguments instead of 2. Here is an example of a cross-version solution: https://github.com/thierry-martinez/metapp/commit/713257f9be692534d82219c21505be504571087b

hanshoglund commented 1 year ago

Made a PR to reflect the current state https://github.com/upenn-cis1xx/camelot/pull/91