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

PR: Fallback to default validator if subschema is provided #198

Merged
merged 3 commits into from
Nov 21, 2020

Conversation

goanpeca
Copy link
Contributor

@goanpeca goanpeca commented Nov 16, 2020

Fixes #197

This fixes a regression introduced when using fastjsonschema, which does not directly support to validate a "reference"/"subschema". For these cases, iter_errors falls back to the default validator that uses json schema.

@goanpeca
Copy link
Contributor Author

goanpeca commented Nov 16, 2020

@MSeal this will need a new release 5.0.9 🙃

Thanks!

@goanpeca
Copy link
Contributor Author

@bollwyvl any thoughts :) ?

Copy link
Member

@rgbkrk rgbkrk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick turnaround!

Copy link
Member

@willingc willingc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @goanpeca.

@willingc willingc mentioned this pull request Nov 21, 2020
6 tasks
@willingc
Copy link
Member

Merging this so folks waiting on #197 fix can pull from HEAD until release 5.1.

Thanks @goanpeca ☀️

@willingc willingc merged commit 0ba4ea5 into jupyter:master Nov 21, 2020
@goanpeca goanpeca deleted the fix/fast-error branch November 21, 2020 21:08
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.

When using fastjsonschema, nbformat.v4.new_code_cell throws error
3 participants