Changeset 47207 for trunk/phpcs.xml.dist

Timestamp:

02/07/2020 07:14:29 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Adjust coding standards to always omit parentheses for include/require statements.

These are language constructs, not function calls, so the parentheses are unnecessary.

This updates the PHPCS configuration file the enforce the sniff until it is moved from the WordPress-Extra ruleset to the WordPress-Core ruleset upstream.

Follow-up to [47198].

Props desrosj, jrf, GaryJ.
Fixes #49376.

File:

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

trunk/phpcs.xml.dist

@@ -170 +170 @@
 
     <!-- Whitelist the I18n functions file from issues identified by the I18n sniff
-        (such as calling the low-level translate() function). -->
+         (such as calling the low-level translate() function). -->
     <rule ref="WordPress.WP.I18n">
         <exclude-pattern>/src/wp-includes/l10n\.php</exclude-pattern>
@@ -190 +190 @@
 
     <!-- Exclude checking of line endings when reporting errors, but fix them when running phpcbf.
-        Git and SVN manage these pretty well cross-platform as "native".
-        Whitelist configuration files. -->
+         Git and SVN manage these pretty well cross-platform as "native".
+         Whitelist configuration files. -->
     <rule ref="Generic.Files.LineEndings">
         <exclude-pattern>/wp-config\.php</exclude-pattern>
@@ -271 +271 @@
     </rule>
 
+    <!-- Enforce no parenthesis for language constructs, and only one space immediately after.
+         Including this rule is temporary until it is moved from the WordPress-Extra ruleset to
+         the WordPress-Core ruleset upstream.
+    -->
+    <rule ref="PEAR.Files.IncludingFile.BracketsNotRequired">
+        <type>warning</type>
+    </rule>
+    <rule ref="PEAR.Files.IncludingFile.UseRequire">
+        <type>warning</type>
+    </rule>
+    <rule ref="PEAR.Files.IncludingFile.UseRequireOnce">
+        <type>warning</type>
+    </rule>
+    <rule ref="Squiz.WhiteSpace.LanguageConstructSpacing"/>
+
     <!-- Assignments in while conditions are a valid method of looping over iterables. -->
     <rule ref="WordPress.CodeAnalysis.AssignmentInCondition.FoundInWhileCondition">