Refactoring some reused code into a method (inside of the routes
mapper) and adding a constant for all the possible scopes.
`fast_xs` support has been removed.  Use 'String#encode(xml: :attr)`.
remove unused variables
Merge pull request #10118 from vipulnsward/remove_unused_var_task

remove unused variables
Bring back ActiveRecord::Base#connection_handler removed by mistake on b37399ab
Add option to Class#class_attribute for skipping the query method
Deprecate ActiveRecord#Base.default_scopes?

See #10107.
Merge pull request #10113 from Agis-/default_scopes_predicate

Deprecate ActiveRecord#Base.default_scopes?

Closes #10107
change some more merge to merge! on new hashes
sprockets 2.9.x doesn't work with uglifier 2.0 yet.

See https://travis-ci.org/rails/rails/jobs/6110543#L1285 for the failure

cc @josh
Fix markdown formatting
Merge pull request #10120 from vipulnsward/change_merge_on_new_hash

change some more merge to merge! on new hashes
Merge pull request #10121 from Agis-/patch-2

Fix markdown formatting
Forgot to lock the uglifier version in our Gemfile

Related with 575838795019f7402992e3bfa83a0c5048cf143d
Merge pull request #10101 from wangjohn/refactoring_routing_mapper

Refactoring some reused code into a method (inside of the routes mapper)
Added a bang to the end of +raise_nested_attributes_record_not_found+
method to signify an exception possibly being raised.
Merge pull request #10123 from wangjohn/added_bang_to_raise_nested_attributes_method

Added a bang to the raise_nested_attributes_record_not_found method
each to each_value; remove unused vars
Merge pull request #10127 from vipulnsward/each_to_each_value

each to each_value; remove unused vars
Avoid iterating over records hash when not necessary

If the reflection scope is not flagged with distinct value, there is no
need to iterate over the records, so we avoid that by doing the check
before iterating rather than inside the iteration block.