As per coding standard documentation at 'https://www.drupal.org/coding-standards/docs', @file tag docblock should not be there in the files that contain a namespaced class/interface/trait, whose file name is the class name with a .php extension.

CommentFileSizeAuthor
#6 remove_file_tag-2872902-6.patch1.75 KBdhruveshdtripathi
#2 Remove_@file_tag-2872902-2.patch599 bytesMunavijayalakshmi

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
StatusFileSize
new Remove_@file_tag-2872902-2.patch599 bytes

Status: Needs review » Needs work

The last submitted patch, 2: Remove_@file_tag-2872902-2.patch, failed testing.

Munavijayalakshmi’s picture

Munavijayalakshmi commented
Status: Needs work » Needs review
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Needs work

There are a bunch of class files in src that need this too, please tackle all instead of one at a time.
http://cgit.drupalcode.org/menu_link_weight/tree/src/MenuLinkWeightServi...

dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs work » Needs review
StatusFileSize
new remove_file_tag-2872902-6.patch1.75 KB

Removed remaining @file tag docblocks from .php files.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Reviewed & tested by the community

Thanks, that leaves the .test file but that is likely left by port to d8.

johnwebdev’s picture

johnwebdev commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.