search

snoyberg / classy-prelude

A typeclass-based Prelude.
108 stars 15 forks source link

Add (<&&>),(<||>) #125

Closed talw closed 8 years ago

talw commented 8 years ago

There was a great suggestion in issue #109. But it seems like it wasn't realized as the issue is 5 months old. So I took it upon myself :)

snoyberg commented 8 years ago

LGTM. @gregwebs do you have any comment on fixities here? I'm particularly bad at making decisions on those.

gregwebs commented 8 years ago

I don't know visitors because I don't create operators. Seems like they should be similar to the non-applicative.

snoyberg commented 8 years ago

Looks like that's what @talw chose here, which seems reasonable to me as well. I'll go ahead and merge it then :)