-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add support for minItems, maxItems and uniqueItems #37
Conversation
e63604e
to
ee3f775
Compare
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.
Thank you, looks good, but but we need to keep consistency and use snake_case
@Bugagazavr agreed, let's use GitHub failed (a bit) committing the suggestions, I'll rebase / squash by hand. |
Fixes the PerceivedComplexity in #call: > lib/grape-swagger/entity/attribute_parser.rb:10:7: C: > Metrics/PerceivedComplexity: Perceived complexity for call is too high. [17/16]
1f10e17
to
22d2272
Compare
@Bugagazavr done, let me know if more changes are necessary. Fixed a few more typos 😱 and DRY'ed code to |
Thank you |
released |
Adds support for
minItems
,maxItems
anduniqueItems
forArrays
(is_array: true
) as described in theSchema Object
of the OpenAPI 2.0 specification.The
grape-entity
documentation mentions that thedocumentation
bubbles up but this is not the case for the above attributes.