As per coding standard documentation at 'https://www.drupal.org/coding-standards/docs#file', @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
#3 2755019-3.patch114.94 KBneerajsingh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neerajsingh created an issue. See original summary.

neerajsingh’s picture

neerajsingh
Primary language English
Location IN 🇮🇳
CreditAttribution: neerajsingh commented
Assigned: Unassigned » neerajsingh
Status: Active » Needs work
neerajsingh’s picture

neerajsingh
Primary language English
Location IN 🇮🇳
CreditAttribution: neerajsingh commented
Assigned: neerajsingh » Unassigned
Status: Needs work » Needs review
FileSize
2755019-3.patch114.94 KB

Attaching the patch here...

nielsvm’s picture

nielsvm CreditAttribution: nielsvm as a volunteer and at Acquia commented

Woah, wasn't aware of this. Added a few new files in my branch to the patch as well.

nielsvm’s picture

nielsvm CreditAttribution: nielsvm as a volunteer and at Acquia commented
Status: Needs review » Fixed 
[8.x-3.x 8a8670c] Issue #2755019 by neerajsingh, nielsvm: Remove @file tag docblock
 Author: neerajsingh <neerajsingh@3202261.no-reply.drupal.org>
 247 files changed, 1233 deletions(-)

I've reviewed and accepted your patch. For clarity, I'm setting this issue to status Fixed as this change will end up in version 8.x-3.0-beta5. If this issue still exists after that release (or in the -dev versions), please change back with further resolution steps.

Thank you very much for your contribution,

Niels

  • nielsvm committed 8a8670c on 8.x-3.x authored by neerajsingh 
    Issue #2755019 by neerajsingh, nielsvm: Remove @file tag docblock

Status: Fixed » Closed (fixed)

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