Skip to content
This repository has been archived by the owner on Oct 18, 2018. It is now read-only.

Support for 4.06 #22

Open
baileyparker opened this issue Jan 16, 2018 · 3 comments
Open

Support for 4.06 #22

baileyparker opened this issue Jan 16, 2018 · 3 comments

Comments

@baileyparker
Copy link

The opam package for ocamllint requires ocaml-version < "4.04.0". Is this due to AST changes? Any chances an update is easy? I'd be willing to do the legwork if this repo is still active.

@emillon
Copy link
Contributor

emillon commented Jan 16, 2018

Hi,

This is indeed related to AST changes. This was written before ocaml-migrate-parsetree (OMP) made it easy to support several versions of the compiler at the same time.

It should be reasonably easy to port to OMP. I should be able to find some time in the next couple weeks to do so.

(I'd like to cut a release anyway to update the rule base, and try a non-ppx approach in order not to depend on a succesful compilation)

Thanks!

emillon added a commit that referenced this issue Jan 17, 2018
This will make it possible to use newer versions of the OCaml compiler.

Closes #22
emillon added a commit that referenced this issue Jan 26, 2018
This will make it possible to use newer versions of the OCaml compiler.

Closes #22
@XVilka
Copy link

XVilka commented Aug 24, 2018

Any updates on making a new release with OMP? And 4.06 + 4.07 targets?

@emillon
Copy link
Contributor

emillon commented Aug 24, 2018

I think #25 was almost ready, not sure why I didn't finish the PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants