Restore compatibility with Alcotest 1.4.0#22
Conversation
The Alcotest 1.4.0 release adds optional arguments to Alcotest.fail. Eta-expansion is necessary to use this function with the continuation printers.
|
Thanks @craigfe. I'm curious seeing the amount of failures in ocaml/opam-repository#18509 (and the API change in alcotest 1.4.0) whether:
But of course I'm as well fine to merge this as is once alcotest 1.4.0 is merged into opam-repository as is. :) |
|
Hi @hannesm. The rev-dep failures in this release were all due to this line of code ( Now that I've gotten this far it seems most efficient for me to make PRs to fix the few packages that still have this line in their dev versions, but I can still drop the |
|
@craigfe as said, I'm fine with pushing 1.4.0 as is, and merging this (and similar PRs) across the mirage repositories. |
CHANGES: * Compatibility with alcotest 1.4.0 (mirage/arp#22 @craigfe) * Minor updates for CI (mirage/arp#23 @hannesm)
The Alcotest 1.4.0 release adds optional arguments to Alcotest.fail. Eta-expansion is necessary to use this function with the continuation printers.