ceastlund
ceastlund
Would it be simpler to limit the proposal to wrapping applications (of normal function names, infix operators, and prefix operators) the same on both sides of an infix operator? Other...
The results look good.
Ping, @aantron? There are further changes to ppxlib that would require bisect_ppx fixes. We can't update further until this PR goes out. Hope you have time to look at it...
+1, I would find this useful.
It has been replaced by `map_with_expansion_context_and_errors`. It will take a little plumbing to propagate the errors the new way, but beyond that it does the same job.
`Ppxlib` exports a `With_errors` submodule to help propagate those bindings via `>>|` and `>>=` operators. Someone familiar with that convention will have to update the code. It's not hard for...
Good point about the version arguments. If you like I'll make a PR to remove those arguments.
For the test failures, it might help if we wrote some tests directly against the History interface. We could then test individual conversions, downgrade and upgrade, against both randomized and...
Sorry for the long delay in response. It's possible core_bench can be made to work on Windows, but at the moment we don't have equivalent bindings for the low-level timing...
I have updated core_bench in our internal repo to drop most of the ppx dependencies, except for the few it actually uses. The public release process should push out the...