Paul-Elliot Anglès d'Auriac
Paul-Elliot Anglès d'Auriac
Ppxlib's extenders can rewrite extension nodes only in a "context-free" way, that is the rewriting cannot depend on the context. In particular, `let _ = [%fmt "!(abc) !(qsd)"]` would need...
Seems it has been fixed by #372.
I submitted a first PR #289 for the first task: writing tests to verify the current behaviour. I found out that actually, there is no distinction between 2.a) and 2.b)....
Thanks for the report! Time allocated to ppxlib has become a scarce ressource, however I'll do my best to investigate this issue. Sorry in advance for the dealy!
Good catch, thanks! I do think it makes sense to have it inside ppxlib. Can you open a PR, or should I go ahead?
I don't think it is directly supported right now. You can register such a transformation as a whole file transformation (the `impl` and `intf` arguments of `Driver.register_transformation`). Ppxlib should make...
I am not sure which part is still a problem: The creation of ast patterns, or the ast traversal? In any case, unfortunately there is no more documentation on either...
Yes, lots of reformat are needed :) - I agree with the use of `class` from es6, - I don't mind the `switch` instead of `if`, though for me they...
Thanks for your report! The `let _ = fun (_ : t) -> ()` line is added by ppxlib in [this function](https://github.com/ocaml-ppx/ppxlib/blob/main/src/deriving.ml#L703). The reason for it is to remove any...
First of all, let me answer the direct questions: > Do you think this would achieve the same effect? > I'm guessing that the ppx transform happens before the compiler...