Also need to deprecate engine= first. #41

Merged
merged 1 commit into from Apr 16, 2012

Conversation

Projects
None yet
2 participants
Contributor

arunagw commented Apr 16, 2012

Just found that Rails test started failing today after new release. I am not sure that how we can go with this. But we need to deprecate engine= also.

https://github.com/rails/rails/blob/master/activesupport/lib/active_support/json/decoding.rb#L26

Contributor

arunagw commented Apr 16, 2012

Closing this. As Rails is now using new API.

rails/rails#5860

arunagw closed this Apr 16, 2012

arunagw reopened this Apr 16, 2012

Contributor

arunagw commented Apr 16, 2012

I have reopen this again. I think for 3.2.X release we can show warning.

sferik merged commit be020a3 into intridea:master Apr 16, 2012

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