- Remove mention about `ActionView::TemplateHandlers` since this
    module was removed by c1304098.
    Change word `subclasses` to `nested classes`.
    See c7408a0e
  - Remove useless sentence "Beginning with Rails 2, the standard extensions
    are `.erb` for ERB (HTML with embedded Ruby), and `.builder` for Builder (XML generator)."

[ci skip] setting :inverse_of works with :as, :through and :polymorphic options, and is needed for bi-directionality with a scope

[ci skip] Remove :conditions opion from association basics guide

This got replaced by scopes.

Instead of providing a configuration option to set the hash function,
switch to SHA-1 for new apps and allow upgrading apps to opt in later
via `new_framework_defaults_5_2.rb`.

Before Rails 4.0, `config.cache_classes` determined whether application
code was eager loaded. The `config.eager_load` option was introduced to
allow the two behaviours to be configured independently, but this
documentation was never updated to reflect that change.

  - Provide full / Fix generated HTML by `form_for` in the example.
  - Provide full params in the example.
  - Express params[:person] in the sentence.

[ci skip]

This reverts commit b106242f.

Thanks to @Datasnuten for the suggestion.

The [template](https://github.com/rails/rails/blob/master/railties/lib/rails/generators/rails/app/templates/config/storage.yml.tt#L10)
that generates the `config/storage.yml` file has the Amazon S3 key specified as `:amazon`, not `:s3`.

The guides should reflect the nomenclature, given that every other service also
has the name of the company as the key (:google, :microsoft).

Like other Basics and Overview guides.

Currently, generation of guide is an error in `5_2_release_notes.html`.

```
$ bunele exec rake guides:generate:html
Generating 5_2_release_notes.md as 5_2_release_notes.html
rails/guides/rails_guides/markdown.rb:44:in `dom_id': undefined method `[]' for nil:NilClass (NoMethodError)
	from rails/guides/rails_guides/markdown.rb:106:in `block (2 levels) in generate_structure'
```

It seems that it is an error because there are multiple
`active-storage` anchors.

Since Active Storage is a Major feature, it is unnecessary to show
CHANGELOGs, so remove from `Incompatibilities` section.

See #31315 for full discussion

[ci skip] Add missing **DO NOT READ THIS FILE ON GITHUB, GUIDES ARE PUBLISHED ON http://guides.rubyonrails.org.**

`BigDecimal.new` has been deprecated in BigDecimal 1.3.3
 which will be a default for Ruby 2.5.

Refer
https://github.com/ruby/bigdecimal/commit/533737338db915b00dc7168c3602e4b462b23503

* This commit has been made as follows:

```
cd rails
git grep -l BigDecimal.new | grep -v guides/source/5_0_release_notes.md | grep -v activesupport/test/xml_mini_test.rb | xargs sed -i -e "s/BigDecimal.new/BigDecimal/g"
```
- `activesupport/test/xml_mini_test.rb`
Editmanually to remove `.new` and `::`

- guides/source/5_0_release_notes.md
This is a Rails 5.0 release notes.

  - Make all `ActiveRecord::Base.find` as link
  - Remove redundant sentences
    "It also adds the additional condition that the object must be in the collection."

  - Add mention about `config.active_record.internal_metadata_table_name`
  - Add mention about `config.active_record.protected_environments`

* Added the direct method to routing guide.