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

Strict Validation Mode Added #64

Merged
merged 2 commits into from
Jun 1, 2015
Merged

Strict Validation Mode Added #64

merged 2 commits into from
Jun 1, 2015

Conversation

umar-muneer
Copy link
Contributor

  1. Added a Strict Validation mode. If set to true, any properties
    specified in config files that are not declared in the schema will
    result in errors. This is to ensure that the schema and the config
    files are in sync. This brings convict further in line with the concept
    of a “Schema”. By default the strict mode is set to false.
  2. Added test cases for strict mode

umar-muneer added 2 commits January 24, 2015 20:08
1. Added a Strict Validation mode. If set to true, any properties
specified in config files that are not declared in the schema will
result in errors. This is to ensure that the schema and the config
files are in sync. This brings convict further in line with the concept
of a “Schema”. By default the strict mode is set to false.
2. Added test cases for strict mode
1. Added a fix for nested validation checks.
2. Modified test case schema and config files.
@neebz
Copy link

neebz commented May 24, 2015

This is a pretty nice feature and would love to get it merged. Any chance for this to happen anytime soon?

cc: @zaach

@madarche
Copy link
Collaborator

I agree it is in line with the concept of "schema" featured in node-convict. SGTM.

@zaach
Copy link
Contributor

zaach commented Jun 1, 2015

👍 I like it.

zaach added a commit that referenced this pull request Jun 1, 2015
Strict Validation Mode Added
@zaach zaach merged commit ceb1917 into mozilla:master Jun 1, 2015
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.

4 participants