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.

Issue fork g2-2774615

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aks22 created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, removed-file-docblock-g2.patch, failed testing.

The last submitted patch, removed-file-docblock-g2.patch, failed testing.

The last submitted patch, removed-file-docblock-g2.patch, failed testing.

urvashi_vora’s picture

Assigned: aks22 » urvashi_vora

Hi,

I will work on this.

urvashi_vora’s picture

I will apply a patch shortly.

urvashi_vora’s picture

Assigned: urvashi_vora » Unassigned
Status: Needs work » Needs review
FileSize
6.19 KB

Hi,

Please review this patch.

Thanks

fgm’s picture

Parent issue: » #3362685: Port to Drupal 10
Raveen Thakur’s picture

Hello guys,

** I have reviewed & implemented the patch #7. Patch #7 seems working fine for me.
** Thank you.

Raveen Thakur’s picture

Status: Needs review » Reviewed & tested by the community

  • fgm committed 972d207b on 8.x-1.x authored by urvashi_vora
    Issue #2774615 by urvashi_vora, aks22: Remove @file tag docblock from...
fgm’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the review. Applied to today's dev HEAD.

Raveen Thakur’s picture

You forget to credit me @fgm

fgm’s picture

Ah, sorry, I used the default choices provided by drupal.org, which apparently doesn't take reviews into account.

Status: Fixed » Closed (fixed)

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