-
Notifications
You must be signed in to change notification settings - Fork 196
feat: add OAS 3.2 rules #2367
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
feat: add OAS 3.2 rules #2367
Conversation
|
| Name | Type |
|---|---|
| @redocly/openapi-core | Minor |
| @redocly/cli | Minor |
| @redocly/respect-core | Minor |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
|
454e1ec to
31d7615
Compare
9f0cd9f to
1774ac5
Compare
…ing, and example-values OAS 3.2 rules
1774ac5 to
3d96d21
Compare
| examples: | ||
| first: | ||
| externalValue: "https://example.com/example.json" | ||
| externalValue: https://localhost/example.json |
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 should fix the long running tests issue.
| Root ==> Paths --> PathItem --> Operation --> Parameter --> Schema | ||
| PathItem --> Parameter | ||
| NamedParameter --> Parameter |
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 think, here is missing renaming from NamedParameter -> NamedParameters. This applies to the v1 and v2 docs.
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.
Good catch. Thanks.
What/Why/How?
Added new OAS 3.2 rules:
spec-no-invalid-encoding-combinationserrorwarnerrorspec-discriminator-defaultMappingwarnofferrorspec-example-valueserrorofferrorAlso, I'm going to deprecate the
no-example-value-and-externalValueas theexample-valuescovers its logic and is more versatile.Fixed a wrong NodeType
https://github.com/Redocly/redocly-cli/pull/2367/files#diff-4a44139d5368084db84e0a7c0e98ae999796c0e03e2dc556dbc136a9ce1c4a0e
Updated docs
Reference
Addresses #2316 (comment)
Related: #2363
Testing
Screenshots (optional)
1
2
3
Check yourself
Security