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

Pattern issue #14

Open
wants to merge 5 commits into
base: release
Choose a base branch
from
Open

Pattern issue #14

wants to merge 5 commits into from

Conversation

bldrvnlw
Copy link

@bldrvnlw bldrvnlw commented Aug 3, 2018

Patterns checking was interfering with the object().type() checks. object().type() worked when the type had no keys (E.g. RegExp in the tests) but failed on a user defined type. Bug is corrected and a new test added.

@bldrvnlw
Copy link
Author

bldrvnlw commented Aug 7, 2018

Also added arity checking to the func() schema.

…atches. Added a new flag, _isType, to the compiled schema to denote this situation and a new makeTypeCheck along with changes to the makeSchemaBuilder to set the flag when appropriate. If _isType is present the schema.checks are applied but the keys and pattern checks can be skipped if the correct object instance is present.
…ternatives). package-lock.json added by latest npm
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

1 participant