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

feat: support for "joi.options.presence = required" #49

Merged
merged 2 commits into from
Jun 12, 2019

Conversation

simlu
Copy link
Contributor

@simlu simlu commented Jun 6, 2019

No description provided.

@simlu
Copy link
Contributor Author

simlu commented Jun 6, 2019

Fixes #48

I've also updated dev dependencies (lodash and joi) and fixed tests (very minor changes)

@simlu
Copy link
Contributor Author

simlu commented Jun 7, 2019

@lightsofapollo Would you mind taking a look? We'd really appreciate if this could get merged in

@simlu
Copy link
Contributor Author

simlu commented Jun 10, 2019

@lightsofapollo Sorry for the continuous nudging. I'm sure you're busy, but we'd really appreciate if you could take a quick look.

@lightsofapollo
Copy link
Owner

Back from vaca! Looking right now.

@lightsofapollo
Copy link
Owner

Nagging appreciated !

@lightsofapollo lightsofapollo merged commit 87e5d81 into lightsofapollo:master Jun 12, 2019
@lightsofapollo
Copy link
Owner

Released in 5.1.0

@simlu
Copy link
Contributor Author

simlu commented Jun 12, 2019

Sweet, thank you very much!

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