Skip to content

Commit 489029a

Browse files
authored
Merge pull request #79 from magento/minor-fixes
Reduce false-positive findings
2 parents 8c978ca + af55901 commit 489029a

File tree

6 files changed

+4
-307
lines changed

6 files changed

+4
-307
lines changed

Magento2/Sniffs/Performance/EmptyCheckSniff.php

-141
This file was deleted.

Magento2/Tests/Performance/EmptyCheckUnitTest.inc

-105
This file was deleted.

Magento2/Tests/Performance/EmptyCheckUnitTest.php

-51
This file was deleted.

Magento2/ruleset.xml

+2-8
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@
120120
<rule ref="Magento2.Exceptions.DirectThrow">
121121
<severity>8</severity>
122122
<type>warning</type>
123+
<exclude-pattern>*/Test/*</exclude-pattern>
123124
</rule>
124125
<rule ref="Magento2.Exceptions.ThrowCatch">
125126
<severity>8</severity>
@@ -213,10 +214,6 @@
213214
<severity>7</severity>
214215
<type>warning</type>
215216
</rule>
216-
<rule ref="Magento2.Performance.EmptyCheck">
217-
<severity>7</severity>
218-
<type>warning</type>
219-
</rule>
220217
<rule ref="Magento2.Strings.StringConcat">
221218
<severity>7</severity>
222219
<type>warning</type>
@@ -302,6 +299,7 @@
302299
<rule ref="PSR1.Files.SideEffects">
303300
<severity>6</severity>
304301
<type>warning</type>
302+
<exclude-pattern>*/Test/*</exclude-pattern>
305303
</rule>
306304
<rule ref="PSR1.Methods.CamelCapsMethodName">
307305
<severity>6</severity>
@@ -449,10 +447,6 @@
449447
<rule ref="Squiz.Commenting.ClassComment.TagNotAllowed">
450448
<severity>0</severity>
451449
</rule>
452-
<rule ref="Squiz.Commenting.FunctionCommentThrowTag">
453-
<severity>5</severity>
454-
<type>warning</type>
455-
</rule>
456450
<rule ref="Squiz.PHP.CommentedOutCode">
457451
<properties>
458452
<property name="maxPercentage" value="80"/>

composer.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"AFL-3.0"
77
],
88
"type": "phpcodesniffer-standard",
9-
"version": "1.0.0",
9+
"version": "1.0.1",
1010
"require": {
1111
"php": ">=5.6.0",
1212
"squizlabs/php_codesniffer": "~3.3.0"

composer.lock

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)