Skip to content

Add #valid? and #reason to Domain HASH_METHODS #95

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

Merged
merged 1 commit into from
May 14, 2018

Conversation

parkr
Copy link
Contributor

@parkr parkr commented May 13, 2018

These will now be printed out when you call #to_hash or #to_json on a Domain object.

It seemed like useful information to include, especially the #reason!

@parkr parkr requested a review from benbalter May 13, 2018 23:37
@benbalter benbalter merged commit a3209d6 into master May 14, 2018
@benbalter benbalter deleted the add-valid-reason-to-hash branch May 14, 2018 14:18
@benbalter
Copy link
Contributor

Not enough ☕️. Meant to approve but clicked the wrong green button. Great suggestion!

@parkr
Copy link
Contributor Author

parkr commented May 14, 2018

@benbalter THank you! No sweat. I have another change coming down the pipeline then I'll release a new version 😄

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