-
Notifications
You must be signed in to change notification settings - Fork 7
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
Allow passing options through to Ajv #29
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I'm definitely game to add the ajvOptions parameter.
Do you mind splitting this into 2 PRs – one that adds the ajvOptions parameter, which I'll be happy to take ASAP – and one that changes the type on AdditionalProperties? I want to discuss that change a little more, since I'm not sure I want to explicitly only allow false
for that type.
Thanks for the quick response! I just split it up (but I left the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! And thanks for catching the package-lock.json thing...
I would like to be able to pass options through to Ajv (here is the doc about the available options. For my particular use case, I want to set
verbose: true; removeAdditional: true, allErrors: true