-
Notifications
You must be signed in to change notification settings - Fork 91
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(ibm-required-array-properties-in-response): add new validator rule #684
Conversation
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.
Looks good! Left some minor, optional suggestions
packages/ruleset/src/rules/required-array-properties-in-response.js
Outdated
Show resolved
Hide resolved
packages/ruleset/src/functions/required-array-properties-in-response.js
Outdated
Show resolved
Hide resolved
packages/ruleset/src/functions/required-array-properties-in-response.js
Outdated
Show resolved
Hide resolved
packages/ruleset/src/functions/required-array-properties-in-response.js
Outdated
Show resolved
Hide resolved
This commit introduces the new 'ibm-required-array-properties-in-response' validator rule, which enforces API Handbook guidance related to array properties defined within response schemas. This rule will look for and warn about any array properties defined as optional within each response body's schema (and any sub-schemas reachable from there). Signed-off-by: Phil Adams <[email protected]>
ccfa072
to
1460cdd
Compare
# @ibm-cloud/openapi-ruleset-utilities [1.4.0](https://github.com/IBM/openapi-validator/compare/@ibm-cloud/[email protected]...@ibm-cloud/[email protected]) (2024-09-24) ### Features * **ibm-required-array-properties-in-response:** add new validator rule ([#684](#684)) ([7cc4be2](7cc4be2))
🎉 This PR is included in version 1.4.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
# @ibm-cloud/openapi-ruleset [1.23.0](https://github.com/IBM/openapi-validator/compare/@ibm-cloud/[email protected]...@ibm-cloud/[email protected]) (2024-09-24) ### Features * **ibm-required-array-properties-in-response:** add new validator rule ([#684](#684)) ([7cc4be2](7cc4be2)) ### Dependencies * **@ibm-cloud/openapi-ruleset-utilities:** upgraded to 1.4.0
🎉 This PR is included in version 1.23.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
# ibm-openapi-validator [1.24.0](https://github.com/IBM/openapi-validator/compare/[email protected]@1.24.0) (2024-09-24) ### Features * **ibm-required-array-properties-in-response:** add new validator rule ([#684](#684)) ([7cc4be2](7cc4be2)) ### Dependencies * **@ibm-cloud/openapi-ruleset:** upgraded to 1.23.0
🎉 This PR is included in version 1.24.0 🎉 The release is available on npm package (@latest dist-tag) Your semantic-release bot 📦🚀 |
PR summary
This commit introduces the new 'ibm-required-array-properties-in-response' validator rule, which enforces API Handbook guidance related to array properties defined within response schemas.
This rule will look for and warn about any array properties defined as optional within each response body's schema (and any sub-schemas reachable from there).
PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:
Checklist for adding a new validation rule: