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

Default value #9

Merged
merged 2 commits into from
Apr 13, 2017
Merged

Default value #9

merged 2 commits into from
Apr 13, 2017

Conversation

tbrand
Copy link
Collaborator

@tbrand tbrand commented Apr 13, 2017

#8

@danielpclark
Copy link
Collaborator

danielpclark commented Apr 13, 2017

This looks good to me 👍

If we want to change nil defaults that can be another issue/PR

@tbrand tbrand mentioned this pull request Apr 13, 2017
@tbrand tbrand merged commit c173bf6 into master Apr 13, 2017
@tbrand tbrand deleted the default-value branch April 13, 2017 08:17
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

2 participants