Ignore coding standards in sites/* PHP files as they are for configuration and not real code

There's a wider issue of applying coding standards declared in ./core to things outside of ./core such as this file. https://github.com/alexpott/d8githooks will blindly test any PHP file against the coding standards ignoring its location. We have 3 PHP files outside of ./core to think about:

• sites/default/default.settings.php
• sites/example.settings.local.php
• sites/example.sites.php

And currently we have the following violations:

 8.5.x  alex: ~/dev/drupal/core > phpcs ../sites/

FILE: /Volumes/devdisk/dev/drupal/sites/default/default.settings.php
----------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 7 LINES
----------------------------------------------------------------------
 476 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 477 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 478 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 479 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 480 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 481 | ERROR | [x] Line indented incorrectly; expected 0 spaces,
     |       |     found 2
 483 | ERROR | [x] Line indented incorrectly; expected 1 spaces,
     |       |     found 0
----------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: /Volumes/devdisk/dev/drupal/sites/example.settings.local.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 34 | ERROR | [x] Namespaced classes/interfaces/traits should be
    |       |     referenced with use statements
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 121ms; Memory: 6Mb

We could decide that the nature of these files means that we should just add

// @codingStandardsIgnoreFile

At the top. As nothing is actually wrong. Our coding standards are not really for code include in the main execution of the file like

assert_options(ASSERT_ACTIVE, TRUE);
\Drupal\Component\Assertion\Handle::register();

My vote is just to add

// @codingStandardsIgnoreFile

And be done.

I agree with @alexpott that we should just ignore coding standards.

Attached is patch that does that.

Oh, I didn't see that this was Novice when I started working on it. Sorry for stealing this issue.

@borisson_ patch is applied cleanly. Considering suggestion by @alexpott, its worth that we can ignore coding standard here. Moving to RTBC, open for more suggestion and if need to move to N/W N/R.

Committed to 8.5.x. Thanks! I originally also backported it to 8.4.x, but on second thought I came to the conclusion that it wasn't worth the settings.php changes for the production branch (since even a comment-only change to those could trigger production sites to redo their settings.php.)