Validation: Add "propertyNames" keyword #172
Conversation
awwright
force-pushed
the
validation-propertyNames
branch
from
3280742 to
f1a7b77
Compare
There was a problem hiding this comment.
You need to add this to the hyper-schema meta-schema as well, plus we need to see how #171 plays out.
Aside from that this looks good to me as long as @epoberezkin approves since it was his issue.
| Note the property name that the schema is testing will always be a string. | ||
| </t> | ||
| <t> | ||
| A missing keyword is the same as an empty schema, or a schema testing that the type is "string". |
There was a problem hiding this comment.
Same objection to this language as in PR #171, so this should be adjusted based on the resolution of that PR.
There was a problem hiding this comment.
I can say "A missing keyword exhibits the same behavior as an empty schema" or something like that... The thing is it has to be informative, since it's noting something that's already true.
awwright
force-pushed
the
validation-propertyNames
branch
from
f1a7b77 to
40646a6
Compare
|
I just removed the paragraph, we can figure out what to do in #171 regarding "default" language |
|
at least something useful from me, not just the stream of disruption ;) |
Language for #70