-
-
Notifications
You must be signed in to change notification settings - Fork 286
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
Fix links
not adhering to its meta-schema given invalid dependencies
#1576
base: draft-04
Are you sure you want to change the base?
Fix links
not adhering to its meta-schema given invalid dependencies
#1576
Conversation
In Draft 4, `dependencies` is either a schema or an array of unique strings. See: https://json-schema.org/draft-04/draft-fge-json-schema-validation-00#rfc.section.5.4.5 See: https://www.learnjsonschema.com/draft4/validation/dependencies/ Signed-off-by: Juan Cruz Viotti <[email protected]>
i don't see this file in |
@jeremyfiel I'm targeting Draft 4 though. |
I suppose that's true But these changes should probably go to the hyper schema repo linked |
I don't understand what you're asking for here. The definition of
Source: https://json-schema.org/draft-04/schema#/properties/dependencies This is consistent with the language of the specification: an object where each property (individually) must be a schema or an array of strings. |
Correct, but on the schema I'm linking, it is set to just a string, and not an array of strings? |
@jviotti Good catch! A string was a valid value for |
In Draft 4,
dependencies
is either a schema or an array of unique strings.See: https://json-schema.org/draft-04/draft-fge-json-schema-validation-00#rfc.section.5.4.5
See: https://www.learnjsonschema.com/draft4/validation/dependencies/