Skip to content

Add Alt instance#2

Merged
paf31 merged 1 commit intomasterfrom
alt
Aug 20, 2014
Merged

Add Alt instance#2
paf31 merged 1 commit intomasterfrom
alt

Conversation

@paf31
Copy link
Contributor

@paf31 paf31 commented Aug 20, 2014

@garyb I need this for purescript-yargs. Can you please review?

@garyb
Copy link
Member

garyb commented Aug 20, 2014

👍

There's no Plus or Alternative for Either is there?

@paf31
Copy link
Contributor Author

paf31 commented Aug 20, 2014

You can make some message up for empty, but I think it's a bit hacky.

@paf31
Copy link
Contributor Author

paf31 commented Aug 20, 2014

Shall I merge?

@garyb
Copy link
Member

garyb commented Aug 20, 2014

Yep, go for it. 🍖

paf31 added a commit that referenced this pull request Aug 20, 2014
@paf31 paf31 merged commit 141808c into master Aug 20, 2014
@paf31 paf31 deleted the alt branch August 20, 2014 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants