Skip to content

WIP: adds Maybe monad #89

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 16, 2019
Merged

WIP: adds Maybe monad #89

merged 3 commits into from
Jun 16, 2019

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Jun 15, 2019 •
edited
Loading

I have reintroduced typed Maybe monad.

Things to do later:

  • Docs about Maybe
  • Readme update
  • New @pipeline example with Maybe
  • New composition rules, @astynax your advice is appreciated
  • I will move @pipeline and is_successful to a separate module
  • I will retype is_successful to work with Maybe, and add tests for this case

@astynax please, have a look.

@sobolevn sobolevn requested a review from astynax June 15, 2019 12:37
@sobolevn sobolevn mentioned this pull request Jun 15, 2019
Closed
@sobolevn sobolevn merged commit cf36f26 into master Jun 16, 2019
@sobolevn sobolevn deleted the issue-25 branch July 1, 2019 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astynax astynax Awaiting requested review from astynax

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sobolevn