union clashes with Data.List import about reactive-banana HOT 4 CLOSED

Unlike filter, the function union is not in the Prelude, so you have to write import Data.List anyway and it's easier to add a qualified keyboard or hide imported functions.

I would really like to avoid giving prefixes to functions that are essentially the same.

Dropping union altogether in favor of the Monoid class would work, but this would make the documentation harder to understand. Sometimes, it's nice to have an explicit name for the monoid operation.

from reactive-banana.

(Oops, accidentally closed the issue by pressing the wrong button.)

from reactive-banana.

Yes, it is ugly to have a random subset of functions end with E. The only palatable thing I can think of of that form would be to suffix everything with E or B, and that's hardly better.

I agree that it can be nice to have a name for the Monoid operation, though the same documentation problem does exist with Behavior's Applicative instance. (I might be more on the side of Monoid if mappend was called (<>) :))

This isn't really a big deal, of course; just something I ran into unexpectedly. I'll leave the issue for you to close if you think it's fine.

(I keep accidentally closing issues, too.)

from reactive-banana.

Oh well, it looks like there's no perfect solution at the moment. I'll close this for now, maybe the future will bring some insights. But thanks for bringing it up anyway. :-)

from reactive-banana.