Skip to content

Commit 60b4af5

Browse files
authored
Merge pull request #127 from humanmade/fix-multiple-arguments
Restore `allowMultipleArguments`
2 parents 94afa2e + adfc20b commit 60b4af5

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

HM/ruleset.xml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,25 @@
8989

9090
<rule ref="WordPress.WP.TimezoneChange" />
9191

92+
<!--
93+
Restore the ability to have multiple arguments per line
94+
95+
WPCS disallowed this behavior in 1.1.0, but we'd like to keep it until
96+
there is a reason to disallow multiple arguments.
97+
Ref: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/commit/bb8a48671e213a5588a6439ea52411eeefab4b0f
98+
-->
99+
<rule ref="PEAR.Functions.FunctionCallSignature">
100+
<properties>
101+
<property name="allowMultipleArguments" value="true"/>
102+
</properties>
103+
</rule>
104+
<rule ref="PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket">
105+
<severity phpcs-only="true">0</severity>
106+
</rule>
107+
<rule ref="PEAR.Functions.FunctionCallSignature.CloseBracketLine">
108+
<severity phpcs-only="true">0</severity>
109+
</rule>
110+
92111
<!--
93112
HM Rules / HM RULEZZZZ
94113

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"license": "GPL-2.0-or-later",
66
"require": {
77
"php": ">=7.1",
8-
"wp-coding-standards/wpcs": "^1.0.0",
8+
"wp-coding-standards/wpcs": "1.2.1",
99
"automattic/vipwpcs": "^0.4.0",
1010
"fig-r/psr2r-sniffer": "^0.5.0",
1111
"squizlabs/php_codesniffer": "~3.4.0",

0 commit comments

Comments
 (0)