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

Fix Ruby 2.7 keyword params deprecation warning #939

Merged

Conversation

mattbrictson
Copy link
Contributor

@mattbrictson mattbrictson commented Apr 11, 2020

This fixes the following warnings that are printed when using friendly_id with Ruby 2.7.1:

Using friendly_id @ master:

friendly_id-984dac788d10/lib/friendly_id/history.rb:75: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
rails-370188aea03f/activerecord/lib/active_record/associations.rb:1370: warning: The called method `has_many' is defined here

Using this PR, no warnings are printed.

@mattbrictson
Copy link
Contributor Author

The mysql CI failures seem to be unrelated to this PR.

@parndt
Copy link
Collaborator

parndt commented Apr 13, 2020

@mattbrictson I've merged in #940, could you please rebase this PR?

@mattbrictson mattbrictson force-pushed the fix-ruby-2.7-deprecation-warnings branch from 64ebe02 to 9d96bf2 Compare April 13, 2020 20:52
@mattbrictson
Copy link
Contributor Author

Rebased ✔️

@parndt parndt merged commit fab2b78 into norman:master Apr 13, 2020
@parndt
Copy link
Collaborator

parndt commented Apr 13, 2020

Thank you!

@mattbrictson mattbrictson deleted the fix-ruby-2.7-deprecation-warnings branch April 13, 2020 21:38
cdccollins added a commit to alphagov/contacts-admin that referenced this pull request Sep 16, 2020
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