As per coding standard documentation at https://www.drupal.org/coding-standards/docs#file, the @file 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
#2 2864582-2.patch2.67 KBPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2864582-2.patch2.67 KB

Applying the patch, please review

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community

@Pavan B S, Thanks for the patch & it works well as i checked it manually by applying it in module file.

  • lbainbridge committed dd8735b on 8.x-1.x authored by Pavan B S 
    Issue #2864582 by Pavan B S: Remove @file tag docblock from all the .php...
lbainbridge’s picture

lbainbridge CreditAttribution: lbainbridge at Northern Commerce commented
Status: Reviewed & tested by the community » Fixed

Thanks Pavan, committed to dev!

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented

@lbainbridge, Thanks for committing the patch file, but i think you missed to provide me the RTBC credit. Can you please do the needful commit again?

Status: Fixed » Closed (fixed)

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