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

Silence deprecation warning from the rename of Module#parent to Module#module_parent #345

Conversation

agrobbin
Copy link
Contributor

DEPRECATION WARNING: Module#parent has been renamed to module_parent. parent is deprecated and will be removed in Rails 6.1.

Rails 6 deprecated Module#parent and friends in rails/rails#34051, replacing them with a module_*-prefix naming scheme. This checks to see if module_parent is defined, using it if it's available.

…dule#module_parent`

Rails 6 deprecated `Module#parent` and friends in rails/rails#34051, replacing them with a `module_*`-prefix naming scheme. This checks to see if `module_parent` is defined, using it if it's available.
@agrobbin agrobbin changed the title silence deprecation warning from the rename of Module#parent to Module#module_parent Silence deprecation warning from the rename of Module#parent to Module#module_parent Sep 23, 2019
@gee-forr
Copy link

I really hope the CI builds failing are resolved and that this gets merged and released. This will resolve #358

@gee-forr
Copy link

@agrobbin - I know it's been over a year, but is there any chance you would be able to see why the builds are failing and get them passing again?

@ghost ghost mentioned this pull request Jan 11, 2021
@cooperka
Copy link

cooperka commented Feb 1, 2021

FYI this PR can be closed in favor of: #365

Hossam-Hawary added a commit to Hossam-Hawary/activeadmin_addons that referenced this pull request Jun 12, 2022
@kerrizor
Copy link
Collaborator

Thanks, @cooperka

..and sorry, folks, for the long period of silence here - I can only beg "pandemic" as an excuse.

If this is still an issue, feel free to open against 1.7.x

@kerrizor kerrizor closed this Jun 12, 2022
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.

4 participants