allow 'all' for :domain option in addition to :all
Don't remove mailer layouts files
Remove custom errors page section from the guides

This pattern is too problematic and introduces a lot of edge cases:

  * On 4.2, the issue https://github.com/rails/rails/issues/15124 is
    back again.
  * needs to define each action for each http status otherwise the
    router raises ActionController::RoutingError (No route matches).
  * If the router has `match "/*username",...` and some action is
    missing, Rails will pick up the "match" and try to do its job.
  * encourages people to copy & paste programming. Not DRY.

[ci skip]
Fix Sidekiq ActiveJob integration setup

* .connect on a Redis connection wasn't valid
* Reset logger after we're done testing for redis connection to avoid
  "closed stream" error when starting server for real from a fork
Hide potentially sensitive ActiveJob params from logs

* Show GlobalID instead of full object .inspect output
Only filter GIDable objects; remove cosmetic changes
fixing mailer previews for apps with globbing route
Merge pull request #17792 from rockrep/master

allow 'all' for :domain option in addition to :all
We don't need to call to_sym. Just compare the string
Deprecate `Class` handler in `PredicateBuilder`

Users should pass strings to queries instead of classes
Merge pull request #17825 from aripollak/fix-activejob-sidekiq-integration

Fix Sidekiq ActiveJob integration setup

Conflicts:
	activejob/test/support/integration/adapters/sidekiq.rb
Merge pull request #17797 from y-yagi/donot_delete_mailer_layouts

Don't remove mailer layouts files
Merge pull request #17916 from mrgilman/deprecate-class-handler

Deprecate `Class` handler in `PredicateBuilder`
Remove environment dependency for db:schema:load

All of the behavior :environment was giving (that db:schema:load needed)
was provided as well with :load_config.

This will address an issue introduced in
https://github.com/rails/rails/pull/15394. The fact that db:schema:load
now drops and creates the database causes the Octopus gem to have [an
issue](https://github.com/tchandy/octopus/issues/273) during the drop
step for the test database (which wasn't happening in db:schema:load
before). The error looks like:

    ActiveRecord::StatementInvalid: PG::ObjectInUse: ERROR:  cannot drop the currently open database
    : DROP DATABASE IF EXISTS "app_test"

Because of the timing, this issue is present in master, 4-2-*, and
4.1.8.

A note to forlorn developers who might see this: "Additionally" in a
commit message means you should have a separate commit, with a separate
justification for changes. Small commits with big messages are your
friends.
Remove deprecated behavior allowing nested arrays as query values
Merge pull request #17920 from calebthompson/dont-rely-on-environment-task-for-schema-load

Remove environment dependency for db:schema:load
Use try instead of checking for GlobalID::Identification
Correctly handle multiple attribute method prefix/suffixes which match

Active Record defines `attribute_method_suffix :?`. That suffix will
match any predicate method when the lookup occurs in Active Model. This
will make it incorrectly decide that `id_changed?` should not exist,
because it attempts to determine if the attribute `id_changed` is
present, rather than `id` with the `_changed?` suffix. Instead, we will
look for any correct match.
Merge pull request #17919 from mrgilman/stop-supporting-nested-arrays

Remove deprecated behavior allowing nested arrays as query values
Inline to_global_id conversion