-
Notifications
You must be signed in to change notification settings - Fork 184
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 proper groups schema validation #99
Conversation
css/properties.schema.json
Outdated
@@ -12,12 +12,53 @@ | |||
} | |||
} | |||
}, | |||
"stringList": { | |||
"groupList": { | |||
"type": "array", | |||
"minItems": 1, | |||
"uniqueItems": true, | |||
"items": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be "$ref": "definitions.json#/groupList"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing that out! This got actually redundant, because we have the data in definitions.json. So I just removed this item in a2f2373.
Sebastian
It's awesome! Would be cool to add other common enums to |
Yes, that's the goal. This change is just the first step. Sebastian |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is very cool and a great outcome of working together in person!
I will open an issue to do the same for other enums (appliesTo etc). We also need to update the docs, but I'll mention that in the follow-up issue, too.
Added a detailed groups validation to the JSON schema, split it out to a separate file and fixed/updated the groups within the data accordingly.