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

Adding documentation to the Rails initialization guide. #11160

Conversation

wangjohn
Copy link
Contributor

The documentation mainly concerns config/application.rb and the use of initializers in the boot process. Please let me know if you see any mistakes or bad/incorrect wording.

on your needs.
When `require APP_PATH` is executed, `config/application.rb` is loaded (recall
that `APP_PATH` is defined in bin/rails). This file exists in your app and it's
free for you to change based on your needs.

Choose a reason for hiding this comment

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

Extra spaces at the beginning of these lines :)

@carlosantoniodasilva
Copy link
Member

Looks good to me 👍

/cc @steveklabnik @fxn

exists, `bundler/setup` is then required.
exists, then `bundler/setup` is required. The require of `bundler/setup` is
used by bundler to configure the load path for loading the dependencies in
the Gemfile.
Copy link
Member

Choose a reason for hiding this comment

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

I like this! But rather than repeat 'bundler/setup', why not change to this?

This require is used by Bundler to configure the load path for your Gemfile's dependencies.

Note the B in Bundler as well. :)

@steveklabnik
Copy link
Member

This is great, thanks! :D

@lukaszx0
Copy link
Member

It would be good to add [ci skip] to commit message as well.

@wangjohn
Copy link
Contributor Author

Thanks for all the comments! I've incorporated all your feedback and rebased.

vijaydev added a commit that referenced this pull request Jun 28, 2013
…itialization_guide

Adding documentation to the Rails initialization guide.
@vijaydev vijaydev merged commit 572f9b6 into rails:master Jun 28, 2013
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.

None yet

5 participants