Support from Acquia helps fund testing for Drupal Acquia logo

Comments

harsha012 created an issue. See original summary.

harsha012’s picture

Status: Active » Needs review
FileSize
26.29 KB

added a Patch

Status: Needs review » Needs work

The last submitted patch, 2: coding-standard-2825363-1.patch, failed testing.

The last submitted patch, 2: coding-standard-2825363-1.patch, failed testing.

harsha012’s picture

Component: Code » Documentation
Assigned: harsha012 » Unassigned
Category: Bug report » Task
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: coding-standard-2825363-1.patch, failed testing. View results

NickDickinsonWilde’s picture

Status: Needs work » Needs review
FileSize
24.75 KB

Updated patch - just removed a few no longer relevant lines.

NickDickinsonWilde’s picture

Status: Needs review » Reviewed & tested by the community
NickDickinsonWilde’s picture

Just closed #2762649: Remove @file tag docblock from all the .php files. as duplicate. Since it was opened first, should credit Vinay15 on this one when it gets committed as well.

NickWilde credited Vinay15.

NickDickinsonWilde’s picture

Status: Reviewed & tested by the community » Fixed

Adding credit from other issue

  • 515d45f committed on 8.x-2.x
    Issue #2825363 by NickWilde, harsha012, Vinay15: Coding standards
    
NickDickinsonWilde’s picture

Committed, thanks Vinay15 & Harsha012

Status: Fixed » Closed (fixed)

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