Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

getOrElse instance method on either #111

Closed
wants to merge 1 commit into from

Conversation

wennergr
Copy link

@wennergr wennergr commented Jun 6, 2016

:: Either a b ~> b -> b

Returns the right value if the instance is a Right, otherwise the provided
default value will be returned.

:: Either a b ~> b -> b

Returns the right value if the instance is a `Right`, otherwise the provided
default value will be returned.
@davidchambers
Copy link
Member

How about fromEither :: b -> Either a b -> b instead, as proposed in sanctuary-js/sanctuary#130? This would be consistent with S.fromMaybe. Consistency between ramda-fantasy and Sanctuary seems valuable given the goal to merge ramda-fantasy and sanctuary-types (once the latter exists).

@buzzdecafe
Copy link
Member

closing in favor of fromEither as @davidchambers suggested

@buzzdecafe buzzdecafe closed this Aug 23, 2016
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.

3 participants