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

Comparing equality between ActionController::Parameters and a Hash #6

Closed
randomcodenz opened this issue Aug 19, 2017 · 3 comments
Closed
Assignees

Comments

@randomcodenz
Copy link
Owner

Build a classifier to classify the following:

Comparing equality between ActionController::Parameters and a Hash is deprecated and will be removed in Rails 5.1. Please only do comparisons between instances of ActionController::Parameters. If you need to compare to a hash, first convert it using ActionController::Parameters#new. (called from == at /var/lib/jenkins/gems/ruby/2.3.0/gems/rspec-expectations-3.5.0/lib/rspec/matchers/built_in/eq.rb:35)

mrwinton pushed a commit that referenced this issue Aug 20, 2017
@mrwinton
Copy link
Collaborator

Branch equality_comparison_between_parameters_and_hash ready for review.

@randomcodenz
Copy link
Owner Author

Reviewed - see comment in the pull request. Is a hard one though as I don't know how we can label the deprecation warning.

@mrwinton
Copy link
Collaborator

mrwinton commented Sep 5, 2017

Yeah, naming is hard :)

I've updated the deprecation title to "Parameters and Hash comparison" I think once you have looked at one of the deprecations warnings it'll be easy to recall what this title is referring to.

Do you think it's sufficient?

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

No branches or pull requests

2 participants