search

ocaml-ppx / ppx_import

Less redundancy in type declarations and signatures
MIT License
89 stars 29 forks source link

Add test case in interface file (.mli) #61

Closed bbc2 closed 3 years ago

bbc2 commented 3 years ago

This ensures that the PPX is registered for interfaces.

ejgallego commented 3 years ago

Thanks @bbc2 , I wonder why the tests failed, ummm I'll have a look myself.

bbc2 commented 3 years ago

My bad, I had incorrectly assumed that the CI was already broken and that the interface file didn't have anything to do with it.

I've fixed it by using a different module name, which should also be more in line with the rest of the tests. It seems to work on my test CI: