WordPress.org
Get WordPress

Make WordPress Core

Changeset 43348 for trunk/phpcs.xml.dist


Ignore:
Timestamp:
06/15/2018 03:56:58 PM (7 years ago)
Author:
jrf
Message:

Build/Tools: Update PHPCS ruleset for WP Core and related Composer setup.

.gitignore + svn:ignore:

  • Add the typical filenames of overloaded PHPCS configs to .gitignore.

Composer:

  • Use the develop (Packagist dev-master) version of WPCS as it contains lots of bugfixes.
  • Remove the PHPCS dependency. This is a dependency of WPCS, not of WP Core itself. This will also make sure that the PHPCS version used is always one which is supported by WPCS.
  • Refreshed the composer.lock file.

PHPCS ruleset:

  • Removed a reference to a sniff which doesn't exist in WPCS yet.
  • Use the PHPCS 3.x basepath option to clean up the file paths PHPCS shows in the reports.
  • Use the PHPCS 3.x parallel option to enable parallel scanning whenever possible to speed up the scans.
  • Whitelist the wp-includes/l10n.php file from issues being reported by the WordPress.WP.I18n sniff.

Fixes #44366.

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk

    • Property svn:ignore

      •  

        old new  
        33wp-tests-config.php
        44.htaccess
        5 
        65# Files and folders related to build/test tools
        76phpunit.xml
         7phpcs.xml
         8.phpcs.xml
        89node_modules
        910npm-debug.log

  • trunk/phpcs.xml.dist

    r43013 r43348  
    55    <rule ref="WordPress-Core"/>
    66    <rule ref="WordPress.CodeAnalysis.EmptyStatement"/>
    7     <rule ref="WordPress.Commenting.DocblockFormat"/>
    87
    98    <arg name="extensions" value="php"/>
     9
     10    <!-- Strip the filepaths down to the relevant bit. -->
     11    <arg name="basepath" value="./"/>
     12
     13    <!-- Check up to 20 files simultanously. -->
     14    <arg name="parallel" value="20"/>
    1015
    1116    <!-- Show sniff codes in all reports -->
    … …  
    7075    </rule>
    7176
     77    <!-- Whitelist the I18n functions file from issues identified by the I18n sniff
     78        (such as calling the low-level translate() function). -->
     79    <rule ref="WordPress.WP.I18n">
     80        <exclude-pattern>/src/wp-includes/l10n\.php</exclude-pattern>
     81    </rule>
     82
    7283    <rule ref="Generic.Functions.FunctionCallArgumentSpacing">
    7384        <exclude-pattern>/wp-config\.php</exclude-pattern>
Note: See TracChangeset for help on using the changeset viewer.