Changeset 43571 for trunk/phpcs.xml.dist

Timestamp:

08/17/2018 01:50:26 AM (6 years ago)

Author:

pento

Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

• Multiline function calls must now put each parameter on a new line.
• Auto-formatting files is now part of the grunt precommit script.
• Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:

• trunk/phpcs.xml.dist (modified) (3 diffs)

trunk/phpcs.xml.dist

@@ -5 +5 @@
     <rule ref="WordPress-Core"/>
     <rule ref="WordPress.CodeAnalysis.EmptyStatement"/>
+
+    <rule ref="PEAR.Functions.FunctionCallSignature">
+        <properties>
+            <property name="allowMultipleArguments" value="false"/>
+        </properties>
+    </rule>
 
     <arg name="extensions" value="php"/>
@@ -19 +25 @@
     <file>.</file>
 
+    <!-- Exclude the build folder in the current directory, as Travis puts the checkout in a build directory. -->
+    <exclude-pattern type="relative">^build/*</exclude-pattern>
+
     <!-- Directories and third party library exclusions -->
-    <exclude-pattern>/build/*</exclude-pattern>
     <exclude-pattern>/node_modules/*</exclude-pattern>
     <exclude-pattern>/vendor/*</exclude-pattern>
@@ -88 +96 @@
     </rule>
 
+    <rule ref="Generic.Files.LineEndings">
+        <exclude-pattern>/wp-config\.php</exclude-pattern>
+        <exclude-pattern>/wp-config-sample\.php</exclude-pattern>
+    </rule>
+
     <!-- Whitelist test classes for select sniffs. -->
     <rule ref="WordPress.Files.FileName">