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

Better runWith errors for State #152

Merged
merged 1 commit into from
Nov 15, 2017
Merged

Better runWith errors for State #152

merged 1 commit into from
Nov 15, 2017

Conversation

evilsoft
Copy link
Owner

Just a spoonful of Sugar

image

On State the runWith method did not throw if one did not return a Pair. This PR now does that and uses this new error to clean up some more of the error handling for all the methods.
Thinking this is more a Bug Fix then a Breaking change...But we will see hows I Feelz when the release actually goes out.

@evilsoft evilsoft changed the title Better runWith Errors for State Better runWith errors for State Nov 10, 2017
@coveralls
Copy link

coveralls commented Nov 10, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 8bd41ec on state-errors into 8fdf460 on master.

@evilsoft
Copy link
Owner Author

image

@evilsoft evilsoft merged commit dc0331f into master Nov 15, 2017
@evilsoft evilsoft deleted the state-errors branch November 15, 2017 06:05
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