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

Add effect module #1

Merged
merged 3 commits into from
Jul 17, 2017
Merged

Add effect module #1

merged 3 commits into from
Jul 17, 2017

Conversation

andrejewski
Copy link
Owner

@andrejewski andrejewski commented Jul 15, 2017

This PR adds helpers for effects. I am hesitant to add anything but the essentials to this framework's API surface. These effect helpers I believe are foundational to building decomposed apps and assembling them together. To ask users to write these helpers by hand has problems:

  • Errors, as they may not be tested with the scrutiny I test with
  • Inconsistency, as there are many ways to implement the happy path without the edge cases
  • Familiarity is lost, as everyone lacks a shared foundation moving between re-created helpers

For these reasons, I believe this module is worth adding.

@andrejewski andrejewski merged commit 052119a into master Jul 17, 2017
@andrejewski andrejewski deleted the effects branch July 17, 2017 01:32
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.

None yet

1 participant