Skip to content

Strict Validation Mode Added#64

Merged
zaach merged 2 commits intomozilla:masterfrom
umar-muneer:master
Jun 1, 2015
Merged

Strict Validation Mode Added#64
zaach merged 2 commits intomozilla:masterfrom
umar-muneer:master

Conversation

@umar-muneer
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Collaborator

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

@zaach
Copy link
Copy Markdown
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