Bump responders dependency version #230

Merged
merged 1 commit into from Jul 5, 2012

Conversation

Projects
None yet
3 participants
@jayzes
Contributor

jayzes commented Jul 2, 2012

Ran into some issues in a project that were fixed in a newer version of the responders gem. Not sure if there's a specific reason that this is locked back to the 0.6 series, but the tests pass and my app runs fine with the 0.9 series.

@carlosantoniodasilva

This comment has been minimized.

Show comment Hide comment
@carlosantoniodasilva

carlosantoniodasilva Jul 2, 2012

Contributor

@jayzes I think there's no special reason to lock this, but you should be able to update responders in your app with a bundle update responders, since inherited resources relies on responders ~> 0.6 and not ~> 0.6.0. In any case, I think it's fine to bump here anyway, if you say everything is passing ;)

Contributor

carlosantoniodasilva commented Jul 2, 2012

@jayzes I think there's no special reason to lock this, but you should be able to update responders in your app with a bundle update responders, since inherited resources relies on responders ~> 0.6 and not ~> 0.6.0. In any case, I think it's fine to bump here anyway, if you say everything is passing ;)

joelmoss added a commit that referenced this pull request Jul 5, 2012

Merge pull request #230 from jayzes/master
Bump responders dependency version

@joelmoss joelmoss merged commit cc6eb1d into activeadmin:master Jul 5, 2012

@joelmoss

This comment has been minimized.

Show comment Hide comment
@joelmoss

joelmoss Jul 5, 2012

Contributor

thx

Contributor

joelmoss commented Jul 5, 2012

thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment