Nathan Rebours
Nathan Rebours
@patricoferris I think you can go ahead and merge this!
I'm not quite sure I understood what you are trying to do but I'll answer to each of your questions individually. Using ppxlib context free rules you cannot expand: ```ocaml...
I'm closing this, please feel free to re-open if you deem it necessary.
Unfortunately what you suggest is not strictly possible. There are a few things you could do to make it work: 1. You can deal with groups of type declarations, if...
That's a good point. We should definitely report those errors somehow, either by preserving the output in such cases (considering error nodes injection as rewriting and therefore requiring an output)...
Yes, it's right that in this dune workflow for ppx we'd have no use for embedding errors as we don't want to commit those errors obviously so we'd have no...
Yes, as Patrick said, ppxlib should not depend on external libraries as it otherwise prevents them from being able to depend on ppxlib. The `Yojson` dependent parts should go into...
That's awesome! Let me know when this is ready for review!
My impression is that we could simplify this quite a bit by having a lifter exposed that returns a Yojson compatible value. It could be built on top of the...
Sorry for the delay by the way, the end of year holidays + the OCaml 5.03 release took quite a lot of my time!