[wp-trac] [WordPress Trac] #65278: PHPCS scans can fail after running `build:dev`

[WordPress Trac]

#65278: PHPCS scans can fail after running `build:dev`
------------------------------+---------------------
 Reporter:  desrosj           |       Owner:  (none)
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  7.1
Component:  Build/Test Tools  |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:                    |     Focuses:
------------------------------+---------------------

Comment (by desrosj):

 A few thoughts on this:
 - This file is probably not being built into the best location. The folder
 name (`css/dist`) implies that it only contains built CSS files. But this
 `registry.php` file is an outlier.
 - I believe that this file is creating during the `build` script upstream,
 so it would either need to be formatted after being built, or the
 script/tool building it should be updated to align `=>` operators.
 - This file should probably be added to the `ignore` list in
 `phpcs.xml.dist` (and also `phpcompat.xml.dist`) since it's not subject to
 version control. But if that were the case already, this would have been
 suppressed and not flagged to be fixed. If this path is chosen, then a
 full list of files that meet this criteria should be created and they
 should all be treated the same.
 - This file (and any other PHP files copied over wholesale from the
 `gutenberg` repository) should ideally be considered for being added to
 version control for better tracking of changes over time through tools
 like `git bisect`/`git blame`.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/65278#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform