Embed errors in the AST instead of raising about ppx_inline_test HOT 2 OPEN

Note that we decided that we will change the ppxlib behaviour regarding the handling of exceptions, to match the current use of raise_errorf in PPXs.
Catching an exception will no longer stop the rewriting process. So, the example I gave in the original issue is not relevant any more.

However, embedding errors can still have advantages: It allows reporting multiple errors, while still outputting valid AST for the part that were successful. In the case of this PPX, the example could be rewritten as:

let%test ("invalid" [@tags "invalid1", "invalid2"]) = false

which, when raising instead of embedding errors, would only report a single tag error, instead of all.

from ppx_inline_test.

Hello @panglesd. I am an outreachy applicant.

This issue looks like a good first bite into the ppxlib error reporting improvement project.

At the moment, I am in the process of understanding how the system works enough to ask competent questions as the need arise.

I will be reaching out to you both, here and on discord. Looking forward to a great collaboration

Best Regards

from ppx_inline_test.