search

fsprojects / Fleece

Json mapper for F#
http://fsprojects.github.io/Fleece
Apache License 2.0
199 stars 31 forks source link

F#+ and Fleece ParseResult #43

Closed wallymathieu closed 6 years ago

wallymathieu commented 6 years ago

Using Fleece together with F#+ can cause some weirdness, mainly due to Failure being both defined in F#+ and Fleece.

gusty commented 6 years ago

I would remove Failure from both projects, since now we have Result.Error which express intentions.

However F#+ is already stabilized, we'll have to wait until next major release.

We can remove it from Fleece.

wallymathieu commented 6 years ago

In f#+ it's part of Validation, so it makes more sense to keep it there.