mdx is required for non-test as well

[reynir]

Maybe this is not intentional, but none the less it's the case. Alternatively, the mdx stanza in lib/core/dune should be declared for the test target. I don't remember how to do that off the top of my head.

[xhtmlboi]

Thank you very much! I'm quite surprised that mdx isn't with-tested. It's probably changed while trying to publish on OPAM. Is it quickly possible to test locally whether, with these rules, the package passes the OPAM IC?

[reynir]

Sorry, I don't understand the question

[xhtmlboi]

My bad.
I've never done a release on OPAM, they've been addressed by @dinosaure and @xvw, but I remember @xvw having trouble (because he's as bad at deploying things as I am :D) with the ppx-expect and mdx contexts. So my question is ‘can we quickly validate that with these changes, publishing to OPAM will still work’?

[reynir]

Sorry, I think I misunderstood dune's behavior. It is already attached to the runtest alias according to the dune docs https://dune.readthedocs.io/en/stable/reference/dune/mdx.html

I think I got confused because I issued dune build and forgot that builds tests as well. And this "test" is located in lib/core/...

[xhtmlboi]

I don't understand much about Dune either (and I'd be delighted to have a more digestible tutorial than the documentation).