Added missing test case for migrations when using sql structure
Remove unused require

We're not longer using `ipaddr` in schema dumper
rm `same_host?`.  The same conditional is two lines down.
Revert "rm `same_host?`.  The same conditional is two lines down."

This reverts commit 79469b4b0c05a50e19699bc9b568042add2d4987.
rm `same_host?`.  The same conditional is two lines down.
Pluck should work with columns of the same name from different tables

The column name given by the adapter doesn't include the table
namespace, so going through the hashed version of the result set causes
overridden keys.

Fixes #15649
Merge pull request #15652 from sgrif/pluck-multiple-named-columns

Pluck should work with columns of the same name from different tables
these methods are always called with a tld_parameter

remove the default parameter since the methods are always called with a
parameter
Updating argument names in default matcher for compiling files

to match arguments used in sprockets-rails [ci skip]
remove useless nil check

irb(main):004:0> /foo/ !~ nil
=> true
irb(main):005:0> /foo/ !~ 'bar'
=> true
irb(main):006:0> /foo/ !~ 'foo'
=> false
There are 3 caveats to Local Precompilation including 1 config change

not 2 caveats with one of them being 2 config changes [ci skip]
pull the port out of the options hash once
scheme should contain one or more characters
only look up the subdomain once
remove useless to_param call

extract_subdomain always returns a string, and to_param calls to_s on a
string
remove unnused method from test
Merge pull request #15656 from JohnRiv/guides_asset_pipeline_precompiling

[ci skip] Updating argument names in default matcher for compiling files
Merge pull request #15657 from JohnRiv/guides_asset_pipeline_local

[ci skip] There are 3 caveats to Local Precompilation including 1 config change
Merge pull request #15655 from fabianrbz/remove-unused-method-in-test

remove unnused method from test
Merge pull request #15647 from sgrif/sg-unused-require

Remove unused require