Marcello Seri
Marcello Seri
Please rebase it on master. There seem to be spurious changes from an old eio-3 branch (like the shell-eio..nix file
I can probably do it in a couple of days if that is ok. I don’t know if it is the best way, but I usually do it manually by...
There is some race in the tests: ``` +++ b/_build/.sandbox/fcaf2ed2da7147d68504b5c372cfd680/default/cohttp-eio/tests/test_client.t.corrected @@ -1,6 +1,7 @@ Test Client.get $ test-server & + Fatal error: exception Eio_luv.Luv_error(EADDRINUSE) (* address already in use *)...
Can you update the changelog? @talex5 do you have further comments?
I am still not fully convinced about the read, but I think we can merge and keep iterating on the interface
I would like to release an alpha version of the new cohttp in few weeks, this hopefully will bring more feedback from possible users on the table and help us...
Hi @jrochel can you tell me if this is still a problem with `cohttp.4.0.0` or the current `master`?
This should not be a problem from the next version of cohttp
I think this was resolved by conduit 3, since that has been pulled it is likely still an issue.
Which versions of conduit and cohttp are you testing?