Skip to content

Commit 1fd3aba

Browse files
authored
Merge pull request #1581 from WordPress-Coding-Standards/feature/remove-arbitraryparenthesesspacing-sniff
Core: Defer to upstream `Generic.WhiteSpace.ArbitraryParenthesesSpacing` sniff (PHPCS 3.3.0)
2 parents d9e633a + dd75660 commit 1fd3aba

File tree

5 files changed

+2
-639
lines changed

5 files changed

+2
-639
lines changed

WordPress-Core/ruleset.xml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -258,11 +258,9 @@
258258
<!-- Rule: Similarly, there should be no space before the colon on return type declarations. -->
259259

260260
<!-- Covers rule: Unless otherwise specified, parentheses should have spaces inside of them. -->
261-
<!-- Duplicate of upstream. Should defer to upstream version once minimum PHPCS requirement has gone up.
262-
https://github.com/squizlabs/PHP_CodeSniffer/pull/1701 -->
263-
<rule ref="WordPress.WhiteSpace.ArbitraryParenthesesSpacing">
261+
<rule ref="Generic.WhiteSpace.ArbitraryParenthesesSpacing">
264262
<properties>
265-
<property name="spacingInside" value="1"/>
263+
<property name="spacing" value="1"/>
266264
<property name="ignoreNewlines" value="true"/>
267265
</properties>
268266
</rule>

WordPress/Sniffs/WhiteSpace/ArbitraryParenthesesSpacingSniff.php

Lines changed: 0 additions & 250 deletions
This file was deleted.

0 commit comments

Comments
 (0)