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

Change wrong usage in documentation about autoload_paths [ci skip] #20759

Merged
merged 1 commit into from
Jul 27, 2015
Merged

Change wrong usage in documentation about autoload_paths [ci skip] #20759

merged 1 commit into from
Jul 27, 2015

Conversation

meinac
Copy link
Contributor

@meinac meinac commented Jul 2, 2015

We can't use this configuration outside of the application.rb

We can't use this configuration outside of the application.rb
@kaspth
Copy link
Contributor

kaspth commented Jul 2, 2015

Is that true, @fxn?

@meinac
Copy link
Contributor Author

meinac commented Jul 2, 2015

@kaspth you can try it by yourself. Just try to add new path to the autoload_paths in environment specific initialiser(it does not matter which environment you use) it will raise error while booting the application.

@zzak
Copy link
Member

zzak commented Jul 2, 2015

I believe this is because the paths get frozen after the app boots.

@meinac
Copy link
Contributor Author

meinac commented Jul 2, 2015

Yes, this is why we can't use it in environment specific initializer. Also freezing this makes sense because even if you change the autocad_paths it does not effect on the loaded paths.

@zzak zzak added the docs label Jul 9, 2015
rafaelfranca added a commit that referenced this pull request Jul 27, 2015
Change wrong usage in documentation about autoload_paths [ci skip]
@rafaelfranca rafaelfranca merged commit ab6f0df into rails:master Jul 27, 2015
rafaelfranca added a commit that referenced this pull request Jul 27, 2015
Change wrong usage in documentation about autoload_paths [ci skip]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants