Result 1.5 - improved 4.08 compatibility

[aantron]

This release mainly addresses janestreet/result#8, which is that Result from result shadows Result from Stdlib, which is available starting from 4.08.

Stdlib.Result has more values declared and a different type name, and all of these become inaccessible because of the shadowing, unless one consistently writes Stdlib.Result instead of just Result. This was a great annoyance if result appeared anywhere in the dependency graph of a project, especially if it was introduced at a late stage in development, which could cause a large amount of code to break.

EDIT: This release makes Result from result include Stdlib.Result on OCaml >= 4.08.

cc @diml

[aantron]

Please consider this as WIP until we address janestreet/result#9 (comment).

[camelus]

Commit: 26d5784

I made an error retrieving the post by the user, sorry about that

☀️ All lint checks passed 26d5784

• These packages passed lint tests: result.1.5

[aantron]

Just repeating the comment from #15893: I believe this can be merged, as only one revdep failure was due to result; that was the test cases of containers.2.7, which were constrained in #15893. There are newer releases of containers, so nothing needs to be done upstream.

[kit-ty-kate]

Thanks (sorry for the huge delay)

[aantron]

Thanks!