sprockets-rails will only add `Rails.application.assets` if compile is
no false. See change on sprockets-rails
https://github.com/rails/sprockets-rails/commit/d7c7ee19991c565eb77ee143be2d009ba4472122

See the behaviour change on sprockets-rails
https://github.com/rails/sprockets-rails/commit/56725e5843662189c6701478f7598db306d5fb02

We need to ignore the `assets_prefix` when running a command like `rake
routes`. However we cannot reference asserts_prefix from action_pack as
that is a sprockets-rails concern.
See this is now implemented on sprockets-rails
https://github.com/rails/sprockets-rails/commit/85b89c44ad40af3056899808475e6e4bf65c1f5a

Removed unused code from request_forgery_protection tests

 Escape README in Plugin builder API documentation

 - So that RDoc will not generate link for README.
 [ci skip]

Uncomment byebug in framework Gemfile

[ci skip] Update ActiveSupport::Subscriber documentation

This reverts commit 6a7cf515.

Correct translate helper docs [ci skip]

No need to capitalize verbs. [CI SKIP]

Follow-up to #19672 and 8b0aa0c1.

[ci skip]

This method is not in use since 479c7cac

These options are not actually available.
Thanks @soutaro for letting me aware of this.

[ci skip] Rework docs, add examples on deprecation proxies

More docs on tagged logging [ci skip]

Fix grammar/style in attribute methods docs

The documentation previously stated that calling `translate(".foo")` was
equivalent to calling `I18n.translate("people.index.foo")` which is
incorrect due to the various other functions of the `translate` view
helper. This has been fixed.

Additionally, a note about forcing the view helper to re-raise
exceptions was added to the section detailing the handling of missing
translations.

Other cleanup includes:

* Consistent formatting of code
* Stop indenting bulleted list as a code sample
* Tighten some of the language
* Wrap at 80 characters.

[ci skip]

[ci skip]

A conjunction was needed to make these sentences correct. Breaking them
up seemed like a better option.

[ci skip]

[ci skip]

follow up to 107526e8

History:

1. `debugger` was there until d2ed433b where it was commented out, and I
   couldn't figure out why

2. Since then we inherited that when switching to `byebug` in 93559da4

Reasons:

1. It's nice to have a debugger when working on framework bugs

2. It's an uncommented dependency in app Gemfiles since Rails 4.2 (see fbe38c9e)

Silence warning from MySQL::Error object in test

When running Active Record MySQL test, this warning is printed in the
console:

    warning: instance variable errno not initialized

It turns out that this is a warning from `mysql` gem in MySQL::Error
object. However, since the `mysql` gem is no longer maintained, and
there won't be a newer version, it make sense for us to just silence
this warning to make the output cleaner.

Remove unused WorkshopsController class in redirect_test.

Closes #19866

[ci skip]