-
Notifications
You must be signed in to change notification settings - Fork 45
feat: accept JSON Schema Draft 6 & Draft 7 #139
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
Conversation
f65812a to
bd7a4d3
Compare
bd7a4d3 to
2eb6acd
Compare
| </div> | ||
| </div> | ||
| </div> | ||
| <div> |
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.
I hope the order doesn't matter for you.
| ]; | ||
|
|
||
| const exampleValidationNames = ['examples', 'example', 'x-example']; | ||
| const exampleValidationNames = ['examples']; |
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.
http-spec will try to produce examples at all times now, thus there's no need to handle example and x-example separately.
| }; | ||
|
|
||
| const validationFormatters: Record<string, (value: unknown) => ValidationFormat | null> = { | ||
| ['const']: createValidationsFormatter('Allowed'), |
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.
json-schema-tree turns const into an enum with a single member
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.
LGTM
|
🎉 This PR is included in version 4.0.0-beta.16 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 4.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Needed by https://github.com/stoplightio/platform-internal/pull/6083
Needs stoplightio/json-schema-tree#8
For stoplightio/elements#1116
See my comments to learn more about the changes.