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

Maybe: Add eliminator for Maybe. #937

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@MostAwesomeDude

MostAwesomeDude commented Jan 15, 2018

This is an essential helper.

Maybe: Add eliminator for Maybe.
This helper is usually called `maybe` or `elimMaybe`.
I don't really have any preference on the name.
@process-bot

This comment has been minimized.

Show comment
Hide comment
@process-bot

process-bot Jan 15, 2018

Thanks for the pull request! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

process-bot commented Jan 15, 2018

Thanks for the pull request! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@mkwiatkowski

This comment has been minimized.

Show comment
Hide comment
@mkwiatkowski

mkwiatkowski Jan 15, 2018

You can already find this helper in the exts package: http://package.elm-lang.org/packages/krisajenkins/elm-exts/27.4.0/Exts-Maybe#maybe Their implementation is a bit shorter: https://github.com/krisajenkins/elm-exts/blob/27.4.0/src/Exts/Maybe.elm#L75 Indeed would be nice to see this in core.

mkwiatkowski commented Jan 15, 2018

You can already find this helper in the exts package: http://package.elm-lang.org/packages/krisajenkins/elm-exts/27.4.0/Exts-Maybe#maybe Their implementation is a bit shorter: https://github.com/krisajenkins/elm-exts/blob/27.4.0/src/Exts/Maybe.elm#L75 Indeed would be nice to see this in core.

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Jan 16, 2018

Member

The policy is to make API requests to the maybe-extra packages. There's probably something like this. If not, they will decide. That's the feeder for core.

Member

evancz commented Jan 16, 2018

The policy is to make API requests to the maybe-extra packages. There's probably something like this. If not, they will decide. That's the feeder for core.

@evancz evancz closed this Jan 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment