Skip to content
This repository has been archived by the owner on May 2, 2020. It is now read-only.

Make validation compatible with voluptuous 0.11 #4

Merged
merged 1 commit into from
Feb 17, 2018

Conversation

balloob
Copy link
Contributor

@balloob balloob commented Feb 17, 2018

opt('created_at') will default to None. This is an invalid value for the coerce_datetime validator. Does work with OPT_DATETIME.

`opt('created_at')` will default to `None`. This is an invalid value for the `coerce_datetime` validator. Does work with `OPT_DATETIME`.
@coveralls
Copy link

coveralls commented Feb 17, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 77ba748 on balloob:patch-1 into 0f6ec60 on armills:master.

@emlove emlove merged commit 5e97d58 into emlove:master Feb 17, 2018
@emlove
Copy link
Owner

emlove commented Feb 17, 2018

👍 Even outside of the voluptuous upgrade, if it is an optional key we should be accepting None anyway.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants