Fix either

[hdgarrood]

Currently, the instances for Either don't record which constructor was used, which means that you get the wrong result if the two types have compatible encodings (for example, Either String String, or Either Number Int).

[garyb]

I noticed this was a little suspect when I revised the Maybe instance a little while ago. I prefer this version as it avoids Alt for branching too, and is explicit about the expected structure, so 👍

[garyb]

Perhaps encoding as an object would be better still?

[hdgarrood]

So like Left 3 -> {"Left": 3}?

[garyb]

Yeah, something like that. Looks like the tests failed for the other one too?

[hdgarrood]

Yeah, I see what the problem is. Oops :) one sec.

[hdgarrood]

Ok, how about this?

[garyb]

Looks great 👍, sorry about the delay.