Skip to content

Commit

Permalink
YoastCS rules: remove exclusion for empty lines
Browse files Browse the repository at this point in the history
The `Squiz.WhiteSpace.SuperfluousWhitespace.EmptyLines` error code flags consecutive blank lines within function declarations.

This rule was previously silenced in YoastCS, but will now be enabled.

Impact on Yoast packages:

| Plugin/Tool       | Errors/Warnings |
|-------------------|-----------------|
| PHPUnit Polyfills | --
| WP Test Utils     | --
| YoastCS           | --
| WHIP              | --
| Yoast Test Helper | 1
| Duplicate Post    | 1
| Yst ACF           | 2
| Yst WooCommerce   | --
| Yst News          | 1
| Yst Local         | 9
| Yst Video         | 23
| Yst Premium       | 20
| Yst Free          | 11
  • Loading branch information
jrfnl committed Nov 20, 2023
1 parent ec67c28 commit 7260b56
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions Yoast/ruleset.xml
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@
<property name="minimum_wp_version" value="6.2"/>
</properties>

<exclude name="Squiz.WhiteSpace.SuperfluousWhitespace.EmptyLines"/>

<!-- No need for this sniff as every Yoast travis script includes linting all files. -->
<exclude name="Generic.PHP.Syntax"/>

Expand Down

0 comments on commit 7260b56

Please sign in to comment.