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

Added keyword name and schema path to keyword usage validation error #1542

Merged
merged 2 commits into from
Apr 11, 2021

Conversation

pcwiek
Copy link
Contributor

@pcwiek pcwiek commented Apr 8, 2021

What issue does this pull request resolve?
resolves #1539

What changes did you make?
Added keyword name & schema path to the reported error message during keyword usage validation.

Is there anything that requires more attention while reviewing?
I'm not entirely sure if we need that one extra test, but I added it anyway to be explicit and actually check the full message and not just parts of it.

@pcwiek
Copy link
Contributor Author

pcwiek commented Apr 8, 2021

Ah, my bad. I have to re-run prettier

@epoberezkin epoberezkin merged commit 6508a03 into ajv-validator:master Apr 11, 2021
@epoberezkin
Copy link
Member

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add the keyword name to the reported error message in internal validation (validateKeywordUsage)
2 participants