Skip to content

Apply values and default of array params to its items #651

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

Merged
merged 2 commits into from
Nov 15, 2017
Merged

Apply values and default of array params to its items #651

merged 2 commits into from
Nov 15, 2017

Conversation

yewton
Copy link
Contributor

@yewton yewton commented Nov 15, 2017

@yewton yewton changed the title Apply values and default of type Array to its items Apply values and default of array params to its items Nov 15, 2017
@coveralls
Copy link

coveralls commented Nov 15, 2017

Coverage Status

Coverage increased (+0.01%) to 97.905% when pulling 28b2458 on yewton:array-of-enums into 8c3fb89 on ruby-grape:master.

@ruby-grape ruby-grape deleted a comment from coveralls Nov 15, 2017
@ruby-grape ruby-grape deleted a comment from coveralls Nov 15, 2017
@ruby-grape ruby-grape deleted a comment from coveralls Nov 15, 2017
@LeFnord
Copy link
Member

LeFnord commented Nov 15, 2017

thanks @yewton :)

@LeFnord LeFnord merged commit 34f38d0 into ruby-grape:master Nov 15, 2017
@yewton yewton deleted the array-of-enums branch November 15, 2017 11:37
hara-y-u pushed a commit to hara-y-u/grape-swagger that referenced this pull request Dec 8, 2017
…#651)

* Apply `values` and `default` of array params to its items

* Fixes ruby-grape#650

https://github.com/ruby-grape/grape#values

* Update CHANGELOG.md
LeFnord pushed a commit to LeFnord/grape-swagger that referenced this pull request Feb 9, 2019
…#651)

* Apply `values` and `default` of array params to its items

* Fixes ruby-grape#650

https://github.com/ruby-grape/grape#values

* Update CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants