Skip to content

Commit d49f669

Browse files
Merge pull request #51 from magento-commerce/imported-magento-magento-coding-standard-255
[Imported] AC-1128: Add severity and type (warning or error) for annotation, htm…
2 parents d753278 + 1b1eae2 commit d49f669

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

Diff for: Magento2/ruleset.xml

+19-5
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,10 @@
144144
<severity>9</severity>
145145
<type>warning</type>
146146
</rule>
147+
<rule ref="Magento2.Html.HtmlBinding">
148+
<severity>9</severity>
149+
<type>warning</type>
150+
</rule>
147151

148152
<!-- Severity 8 warnings: Magento specific code issues and design violations. -->
149153
<rule ref="Magento2.Classes.AbstractApi">
@@ -233,6 +237,10 @@
233237
<severity>8</severity>
234238
<type>warning</type>
235239
</rule>
240+
<rule ref="Magento2.Html.HtmlDirective">
241+
<severity>8</severity>
242+
<type>warning</type>
243+
</rule>
236244

237245
<!-- Severity 7 warnings: General code issues. -->
238246
<rule ref="Generic.Arrays.DisallowLongArraySyntax">
@@ -565,6 +573,15 @@
565573
<severity>6</severity>
566574
<type>warning</type>
567575
</rule>
576+
<rule ref="Squiz.CSS.NamedColours">
577+
<severity>6</severity>
578+
<type>warning</type>
579+
</rule>
580+
<rule ref="Magento2.Less">
581+
<severity>6</severity>
582+
<type>warning</type>
583+
<include-pattern>*\.less$</include-pattern>
584+
</rule>
568585
<rule ref="Squiz.WhiteSpace.SuperfluousWhitespace.StartFile">
569586
<severity>0</severity>
570587
</rule>
@@ -596,16 +613,13 @@
596613
<type>warning</type>
597614
</rule>
598615
<rule ref="Magento2.Annotation">
616+
<severity>5</severity>
617+
<type>warning</type>
599618
<exclude-pattern>*/_files/*</exclude-pattern>
600619
<exclude-pattern>*/Test/*</exclude-pattern>
601620
<exclude-pattern>*Test.php</exclude-pattern>
602621
</rule>
603-
<rule ref="Magento2.Html"/>
604-
<rule ref="Magento2.Less">
605-
<include-pattern>*\.less$</include-pattern>
606-
</rule>
607622
<rule ref="Internal.NoCodeFound">
608623
<severity>0</severity>
609624
</rule>
610-
<rule ref="Squiz.CSS.NamedColours"/>
611625
</ruleset>

0 commit comments

Comments
 (0)