Skip to content

Commit 95ed2d2

Browse files
committed
Ruleset: remove more redundant rules
* The `PEAR.Functions.FunctionDeclaration` sniff can be removed as the PSR12 standard contains the following sniffs which together provide the same safeguards and more: `Squiz.Functions.FunctionDeclaration` and `Squiz.Functions.MultiLineFunctionDeclaration` * The `PEAR.Functions.FunctionCallSignature` sniff can be removed as the PSR12 standard contains the semi-equivalent `PSR2.Methods.FunctionCallSignature` sniff. Note: I'm also removing the exclusions from the standard. IMO, the standard should not have an opinion on this. A custom ruleset for an individual project can bring these exclusions back if/when needed. * The `PEAR.WhiteSpace.ScopeClosingBrace` sniff can be removed as the PSR12 standard contains the semi-equivalent `Squiz.WhiteSpace.ScopeClosingBrace` sniff. * The `PEAR.Classes.ClassDeclaration` sniff can be removed as the PSR12 standard contains the more comprehensive `PSR2.Classes.ClassDeclaration` sniff. * The `PEAR.ControlStructures.ControlSignature` sniff can be removed as the PSR12 standard contains the more comprehensive `Squiz.ControlStructures.ControlSignature` sniff. * The `Zend.Files.ClosingTag` sniff can be removed as the PSR12 standard contains the semi-equivalent `PSR2.Files.ClosingTag` sniff.
1 parent 583b115 commit 95ed2d2

File tree

1 file changed

+0
-20
lines changed

1 file changed

+0
-20
lines changed

PHPParallelLint/ruleset.xml

-20
Original file line numberDiff line numberDiff line change
@@ -30,33 +30,13 @@
3030
#############################################################################
3131
-->
3232

33-
<rule ref="PEAR.Functions.FunctionDeclaration"/>
34-
<rule ref="PEAR.Functions.FunctionCallSignature"/>
35-
<rule ref="PEAR.Functions.FunctionCallSignature.Indent">
36-
<severity>0</severity>
37-
</rule>
38-
<rule ref="PEAR.Functions.FunctionCallSignature.CloseBracketLine">
39-
<severity>0</severity>
40-
</rule>
41-
<rule ref="PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket">
42-
<severity>0</severity>
43-
</rule>
44-
4533
<rule ref="Generic.PHP.NoSilencedErrors"/>
4634
<rule ref="Squiz.PHP.GlobalKeyword"/>
4735
<rule ref="Squiz.PHP.LowercasePHPFunctions"/>
4836
<rule ref="Squiz.PHP.NonExecutableCode"/>
4937

5038
<rule ref="Generic.Strings.UnnecessaryStringConcat"/>
5139

52-
<rule ref="PEAR.WhiteSpace.ScopeClosingBrace"/>
53-
54-
<rule ref="PEAR.Classes.ClassDeclaration"/>
55-
56-
<rule ref="PEAR.ControlStructures.ControlSignature"/>
57-
58-
<rule ref="Zend.Files.ClosingTag"/>
59-
6040
<rule ref="Generic.NamingConventions.ConstructorName"/>
6141

6242
<rule ref="Generic.Metrics.NestingLevel"/>

0 commit comments

Comments
 (0)