Results 190 comments of Raphaël Proust

Ok I had a better look. I didn't notice at first that the bulk of the changes are on the discovery script. The changes seem ok to me. I'll merge...

This seems like a reasonable addition. I'll look into the failure.

I started the other CI tests (I just noticed they needed an approval start) and they paint a much clearer picture: the error occurs when the code is compiled with...

It doesn't work, same error. I'll dig further.

We will do a release in not too long.

@rgrinberg In the cohttp issue discussion you mention > The issue was successfully worked around in the new cohttp client and servers though. Do you have pointers to the fix?...

I haven't really worked on the `Lwt_bytes`/`Lwt_io` part of Lwt. I'll try to have a look some time, but that will be a big context switch so I can't do...

5.1 is not yet available, putting this in draft for now

Should we update to v4? Are there release notes or something that would help finding out if it's relevant?

> Is that documented in the manual/docs? Should code using Lwt switch to `raise`/`reraise`? `reraise` is documented in the doc https://github.com/ocsigen/lwt/blob/master/src/core/lwt.mli#L581 / https://ocaml.org/p/lwt/latest/doc/Lwt/index.html#val-reraise and it is recommended to use in...