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

Support for optional-only map() #41

Merged
merged 2 commits into from Aug 13, 2017

Conversation

mattbishop
Copy link
Contributor

Fixes #40

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 88.091% when pulling 48f0864 on mattbishop:optional-only-map into 264a1c0 on prayerslayer:master.

Repository owner deleted a comment from coveralls Aug 13, 2017
Repository owner deleted a comment from coveralls Aug 13, 2017
Repository owner deleted a comment from coveralls Aug 13, 2017
Repository owner deleted a comment from coveralls Aug 13, 2017
Repository owner deleted a comment from coveralls Aug 13, 2017
Repository owner deleted a comment from coveralls Aug 13, 2017
@prayerslayer
Copy link
Owner

Very nice, thanks a ton! Would you mind adding a test case with an empty map too (the original one from the issue)?

@prayerslayer prayerslayer merged commit e13994d into prayerslayer:master Aug 13, 2017
@prayerslayer
Copy link
Owner

Released as 1.0.0-8, thank you again!

@mattbishop mattbishop deleted the optional-only-map branch August 13, 2017 17:19
@mattbishop
Copy link
Contributor Author

👍

@arichiardi
Copy link
Collaborator

Cool

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

4 participants