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

Remove responders #65

Merged
merged 1 commit into from Feb 18, 2015
Merged

Remove responders #65

merged 1 commit into from Feb 18, 2015

Conversation

nwjsmith
Copy link
Contributor

@nwjsmith nwjsmith commented Dec 9, 2014

Responders are deprecated in Rails 4.2 so we remove the class-level
call to respond_to.

Responders are [deprecated in Rails 4][1] so we remove the class-level
call to respond_to.

[1]: http://edgeguides.rubyonrails.org/4_2_release_notes.html#respond-with-class-level-respond-to
@dlackty
Copy link

dlackty commented Jan 11, 2015

👍 for this!

dewski added a commit that referenced this pull request Feb 18, 2015
@dewski dewski merged commit 851c5fc into peek:master Feb 18, 2015
@nwjsmith nwjsmith deleted the remove-responders branch February 18, 2015 02:36
@studiostark
Copy link

Is there any chance of getting a new gem posted to rubygems? This is a pretty important change for folks upgrading to Rails 4.2.

@dewski
Copy link
Member

dewski commented Jun 23, 2015

Is there any chance of getting a new gem posted to rubygems? This is a pretty important change for folks upgrading to Rails 4.2.

Pushed 0.1.10 up to Rubygems.

@studiostark
Copy link

Thanks!

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.

None yet

4 participants