Skip to content

Core: add section about namespace declaration rules with select new sniffs #2247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 13, 2023
Merged

Core: add section about namespace declaration rules with select new sniffs #2247

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions WordPress-Core/ruleset.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -412,6 +412,29 @@
<!-- Covered by the Squiz.Functions.FunctionDeclarationArgumentSpacing.SpacingAfterReference error code. -->


<!--
#############################################################################
Handbook: Declare Statements, Namespace, and Import Statements - Namespace declarations.
Ref: https://developer.wordpress.org/coding-standards/wordpress-coding-standards/php/#namespace-declarations
#############################################################################
-->
<!-- Rule: Each part of a namespace name should consist of capitalized words separated by underscores. -->

<!-- Rule: Namespace declarations should have exactly one blank line before the declaration and at least one blank line after. -->

<!-- Covers rule: There should be only one namespace declaration per file... -->
<rule ref="Universal.Namespaces.OneDeclarationPerFile"/>

<!-- Rule: ... and it should be at the top of the file.
Note: with only one namespace declaration, it not being at the top of the file would be a parse error. -->

<!-- Covers rule: Namespace declarations using curly brace syntax are not allowed. -->
<rule ref="Universal.Namespaces.DisallowCurlyBraceSyntax"/>

<!-- Covers rule: Explicit global namespace declaration (namespace declaration without name) are also not allowed. -->
<rule ref="Universal.Namespaces.DisallowDeclarationWithoutName"/>


<!--
#############################################################################
Handbook: Object-Oriented Programming - Only One Object Structure (Class/Interface/Trait) per File.
Expand Down