Doesn't work with Merlin about ppx_inline_test HOT 7 CLOSED

I got this library working using jbuilder, but the same issue arises. However, in this case I can't edit the merlin file since it's generated by jbuilder.

I don't think jbuilder should have hard-coded infos about ppx libraries, can this issue be fixed in this project? (for example by somehow detecting the presence of merlin and skipping this error in that case.)

from ppx_inline_test.

You can also just have FLG -ppx 'ppx-jane -as-ppx -inline-test-lib bap' in your .merlin (i.e. without an extra file indirection).

As for having passing the right default info in the META and having a merlin predicate it seems to me like a reasonable idea, we'll look into doing it for the next release (but might not depending on how much it complicates the META).

from ppx_inline_test.

This is now fixed, see https://jbuilder.readthedocs.io/en/latest/tests.html, with the caveat that it may break if one uses more than one library or executable in a the same directory with different flags (ocaml/dune#230).

Cheers everyone!

from ppx_inline_test.

Ah, thanks, I've tried to use the quotes, but delimited arguments incorrectly :)

from ppx_inline_test.

Indeed, I'm closing this ticket then.

The caveat is indeed annoying, I hope that we can solve it one day.

from ppx_inline_test.

Maybe I'm misunderstanding, but this seems to be broken again. I get the same error message as the OP. If I manually add -as-ppx -inline-test-lib bap to my .merlin file, then it works, but it gets overwritten the next time I run dune.

Here's my dune file:

(library
  (name sum)
  (inline_tests)
  (preprocess (pps ppx_inline_test)))

from ppx_inline_test.

Indeed, we discussed this issue here. It was due to a bug in ppxlib, which has now been fixed in master.

from ppx_inline_test.