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

Replace dependentRequired keywords by dependencies #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

richfab
Copy link
Contributor

@richfab richfab commented Sep 25, 2024

Context

The dependentRequired keyword conditionally requires that certain properties must be present if a given property is present in an object. This conditional requirement is used for example to return an error if terms_url is defined but terms_last_updated is missing.

Problem

According to the JSON Schema doc:

Previously to Draft 2019-09, dependentRequired and dependentSchemas were one keyword called dependencies.

Since we're using a version of JSON Schema prior to Draft 2019-09, we must use the keyword dependencies.

What's Changed

This PR replaces dependentRequired keywords by dependencies in system_information.json for v2.3, v3.0 and v3.1-RC.

Before After
The JSON Schema incorrectly returned valid when terms_url was defined but terms_last_updated was missing. The JSON Schema returns an error when terms_url is defined but terms_last_updated is missing (as expected).

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.

1 participant