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

Rails 6 recommends removing require_dependency #255

Open
StefanWallin opened this issue Sep 4, 2019 · 1 comment
Open

Rails 6 recommends removing require_dependency #255

StefanWallin opened this issue Sep 4, 2019 · 1 comment

Comments

@StefanWallin
Copy link

This gem recommends using require_dependency in the initializer to set up bot reloading in development but those should be removed according to the rails 6 upgrade guide using the Zeitwerk autoloader.

It seems to me that maybe only config.autoload_paths is needed nowadays, so maybe we should update the Readme?

@jgorset
Copy link
Owner

jgorset commented Sep 29, 2019

That sounds about right, @StefanWallin! I wonder if it'll be necessary to change the recommended file structure for Zeitwerk to do it's magic, too, though? I think we'll have to verify this before updating the README accordingly. 🤔 Pull requests welcome!

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