xhtmlboi / yocaml

YOCaml is a static site generator, mostly written in OCaml
GNU General Public License v3.0
108 stars 9 forks source link

Remove the conflict with `result.1.5` #48

Closed dinosaure closed 1 month ago

dinosaure commented 11 months ago

From our last release, yocaml must have a conflict with result.1.5 to be installable via opam in any switches (see this commit b58cc7d). This error is not really related to yocaml (as a well known issue from opam-repository's maintainers). @xvw proposed to use Result from preface which should be nice and should fix the initial issue:

A quick fix on YOCaml part can be replacing Result.map_error by Preface.Result.Bifunctor.map_snd. It is a little bit heavy but it should work.

xhtmlboi commented 11 months ago

Even if Bifunctor.map_snd is iso to Result.map_error, I suggest to add to Try a function map_error. I'll take in the following days! Thanks for the release!