Skip to content
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

Allow global settings on Grape::Entity (0.4.x) #167

Merged
merged 1 commit into from
Aug 10, 2015

Conversation

marshall-lee
Copy link
Member

This fixes a regression introduced after merging a #134.
It's something undocumented before but it worked before and I don't see
anything harmful in the presence of this feature.

Fixes #166.

This fixes a regression introduced after merging a ruby-grape#134.
It's something undocumented before but it worked before and I don't see
anything harmful in the presence of this feature.

Fixes ruby-grape#166.
@marshall-lee marshall-lee deleted the fix_global_settings branch August 10, 2015 12:01
@marshall-lee marshall-lee restored the fix_global_settings branch August 10, 2015 12:01
@marshall-lee marshall-lee reopened this Aug 10, 2015
@dblock
Copy link
Member

dblock commented Aug 10, 2015

Merging.

Btw, I think the convention for doing patch releases for a branch name is 0.4.x, NBD.

dblock added a commit that referenced this pull request Aug 10, 2015
Allow global settings on Grape::Entity (0.4.x)
@dblock dblock merged commit 6e0bc05 into ruby-grape:0-4 Aug 10, 2015
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.

2 participants