Skip to content

Commit

Permalink
I grouped rules by Standard (PEAR, Generic, Zend, ...)
Browse files Browse the repository at this point in the history
If a standard bring a rule that is duplicate than the other, we must
keep it commented saying duplicate with.
So at any time we know which rule we match for each standard.
  • Loading branch information
eldy committed Jul 26, 2014
1 parent 3b9cfea commit 9ef3683
Showing 1 changed file with 16 additions and 3 deletions.
19 changes: 16 additions & 3 deletions dev/codesniffer/ruleset.xml
Expand Up @@ -11,10 +11,16 @@

<!-- List of all tests -->


<!-- Rules from Internal Standard -->

<rule ref="Internal.NoCodeFound">
<severity>0</severity>
</rule>


<!-- Rules from Generic Standard -->

<rule ref="Generic.CodeAnalysis.EmptyStatement" />

<rule ref="Generic.CodeAnalysis.EmptyStatement.NotAllowed">
Expand All @@ -24,9 +30,6 @@
<severity>0</severity>
</rule>

<!-- The closing ?> tag MUST be omitted from files containing only PHP. -->
<rule ref="Zend.Files.ClosingTag"/>

<!-- <rule ref="Generic.CodeAnalysis.ForLoopShouldBeWhileLoop" /> -->

<rule ref="Generic.CodeAnalysis.ForLoopWithTestFunctionCall" />
Expand Down Expand Up @@ -153,6 +156,10 @@
<!-- Disabled as this does not support tab -->
<!-- <rule ref="Generic.WhiteSpace.ScopeIndent" /> -->



<!-- Rules from PEAR Standard -->

<rule ref="PEAR.Classes.ClassDeclaration" />

<!-- Check for duplicate class names -->
Expand Down Expand Up @@ -270,4 +277,10 @@
<!-- Already found as a Generic rule -->
<!-- <rule ref="PEAR.WhiteSpace.ScopeIndent" /> -->


<!-- Rules from Zend Standard-->

<!-- The closing ?> tag MUST be omitted from files containing only PHP. -->
<rule ref="Zend.Files.ClosingTag"/>

</ruleset>

0 comments on commit 9ef3683

Please sign in to comment.