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

⚠️ Avoid deprecation warnings on Rails 6 #750

Merged
merged 1 commit into from
May 10, 2019

Conversation

sinsoku
Copy link
Contributor

@sinsoku sinsoku commented May 8, 2019

parent_name will be renamed module_parent_name in Rails 6.1, so it displays deprecation warnings on Rails 6.0.

refs: rails/rails#34051

`parent_name` will be renamed `module_parent_name` in Rails 6.1,
so it displays deprecation warnings on Rails 6.0.

refs: rails/rails#34051
Copy link
Contributor

@delner delner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the heads up on this @sinsoku! Seems pretty straightforward. Will have to add some tests for Rails 6 at some point, but we should be able to add this to an upcoming release.

Your contribution is much appreciated! :)

@delner delner assigned delner and sinsoku and unassigned delner May 8, 2019
@delner delner added bug Involves a bug integrations Involves tracing integrations labels May 8, 2019
@delner delner merged commit 0993eb0 into DataDog:master May 10, 2019
@delner delner added this to the 0.23.2 milestone May 10, 2019
@sinsoku sinsoku deleted the patch/fix_warnings_on_rails6 branch June 14, 2020 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Involves a bug integrations Involves tracing integrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants