Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commite304ee7

Browse files
committed
minor#8600 Add special Abstract prefix not usage for PHPUnit test cases (Soullivaneuh)
This PR was merged into the 3.3 branch.Discussion----------Add special Abstract prefix not usage for PHPUnit test casesAs we can see here:![image](https://user-images.githubusercontent.com/1698357/32507068-bddf178a-c3e6-11e7-8d28-05f3b18ea18c.png)You are not using the `Abstract` prefix at all for the test cases.I assume this is to follow PHPUnit conventions on this part?In this case, the coding standard may be more accurate about this.Commits-------f4d122a Add special Abstract prefix not usage for PHPUnit test cases
2 parentsa7d8d97 +f4d122a commite304ee7

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

‎contributing/code/standards.rst‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,9 +190,10 @@ Naming Conventions
190190

191191
* Use namespaces for all classes;
192192

193-
* Prefix abstract classes with ``Abstract``. Please note some early Symfony classes
194-
do not follow this convention and have not been renamed for backward compatibility
195-
reasons. However all new abstract classes must follow this naming convention;
193+
* Prefix all abstract classes with ``Abstract`` except PHPUnit ``*TestCase``.
194+
Please note some early Symfony classes do not follow this convention and
195+
have not been renamed for backward compatibility reasons. However all new
196+
abstract classes must follow this naming convention;
196197

197198
* Suffix interfaces with ``Interface``;
198199

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp