Skip to content

Commit

Permalink
YoastCS ruleset: remove two sniffs which are now in WPCS
Browse files Browse the repository at this point in the history
WPCS 3.1.0 includes the `Universal.PHP.LowercasePHPTag` (Core) and the `Universal.CodeAnalysis.NoDoubleNegative` (Extra) sniffs, so no need to explicitly require those anymore for YoastCS.
  • Loading branch information
jrfnl committed Apr 4, 2024
1 parent 353a5a3 commit c4216fd
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions Yoast/ruleset.xml
Original file line number Diff line number Diff line change
Expand Up @@ -188,10 +188,6 @@
<!-- PHPCS 3.5.0: This sniff may be added to WPCS in due time and can then be removed from this ruleset. -->
<rule ref="PSR12.Files.OpenTag"/>

<!-- CS: Enforces that a PHP open tag uses lowercase. -->
<!-- PHPCSExtra 1.2.0: This sniff may be added to WPCS in due time and can then be removed from this ruleset. -->
<rule ref="Universal.PHP.LowercasePHPTag"/>

<!-- Demand that "else(if)" is on a new line after the scope closer of the preceding if. -->
<rule ref="Universal.ControlStructures.IfElseDeclaration"/>

Expand All @@ -218,9 +214,6 @@
<!-- CS/QA: Forbid the use of the and/or logical operators. -->
<rule ref="Universal.Operators.DisallowLogicalAndOr"/>

<!-- CS/QA: Forbid the use double `!`. -->
<rule ref="Universal.CodeAnalysis.NoDoubleNegative"/>

<!-- CS: enforce that boolean operators between conditions in multi-line control structures are
consistently at the start or end of the line, not a mix of both. -->
<rule ref="PSR12.ControlStructures.BooleanOperatorPlacement">
Expand Down

0 comments on commit c4216fd

Please sign in to comment.