Skip to content

Commit

Permalink
Merge pull request #2128 from WordPress/feature/core-add-sniffs-for-c…
Browse files Browse the repository at this point in the history
…lass-constant-modifier-order
  • Loading branch information
GaryJones authored Dec 7, 2022
2 parents 1e57c46 + 77de577 commit d8a76b4
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions WordPress-Core/ruleset.xml
Original file line number Diff line number Diff line change
Expand Up @@ -391,14 +391,16 @@
Ref: https://developer.wordpress.org/coding-standards/wordpress-coding-standards/php/#visibility-and-modifier-order
#############################################################################
-->
<!-- Rule: When using multiple modifiers for a class declaration, the order should be as follows:
<!-- Covers rule: When using multiple modifiers for a class declaration, the order should be as follows:
- First the optional abstract or final modifier.
- Next, the optional readonly modifier. -->
<rule ref="Universal.Classes.ModifierKeywordOrder"/>

<!-- Rule: When using multiple modifiers for a constant declaration inside object-oriented structures,
<!-- Covers rule: When using multiple modifiers for a constant declaration inside object-oriented structures,
the order should be as follows:
- First the optional final modifier.
- Next, the visibility modifier. -->
<rule ref="Universal.Constants.ModifierKeywordOrder"/>

<!-- Covers rule: When using multiple modifiers for a property declaration, the order should be as follows:
- First a visibility modifier.
Expand Down

0 comments on commit d8a76b4

Please sign in to comment.