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
#7 2766867-coding-standards.patch19.91 KBDave Reid
#7 interdiff.txt16.25 KBDave Reid
#2 removed-file-docblock-2766867.patch6.05 KBaks22
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aks22 created an issue. See original summary.

aks22’s picture

aks22 CreditAttribution: aks22 as a volunteer and at Cybage Software Pvt Ltd. commented
FileSize
removed-file-docblock-2766867.patch6.05 KB

Please find attached patch.

aks22’s picture

aks22 CreditAttribution: aks22 as a volunteer and at Cybage Software Pvt Ltd. commented
Status: Active » Needs review
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Status: Needs review » Needs work

There are some more redundant @file blocks and coding standards that could be fixed. I'll reroll this shortly with the rest.

Dave Reid credited nehakhadke.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
FileSize
interdiff.txt16.25 KB
2766867-coding-standards.patch19.91 KB

Including more fixes and the rest of the @file fixes.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Status: Needs work » Needs review
Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer and at Lullabot commented
Status: Needs review » Fixed

Committed #7 to 8.x-1.x.

  • Dave Reid committed d1e1721 on 8.x-1.x 
    Issue #2766867 by Dave Reid, aks22, nehakhadke: Fixed coding standards.

Status: Fixed » Closed (fixed)

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