Run coding standards over the entire framework when no major issues are active

Comments

dpi created an issue. See original summary.

dpi’s picture

Assigned: Unassigned » dpi
dpi’s picture

Status: Active » Needs review
StatusFileSize
new85.52 KB

For testbot only.

almaudoh’s picture

We should give credit to @aditya_anurag from the other issue

dpi’s picture

StatusFileSize
new132.94 KB

I reject your proposal :)

This patch has all the manual fixes. Mostly docs I have to create.

dpi’s picture

dpi’s picture

StatusFileSize
new93.68 KB

Attaching initial coder report

  • dpi committed 82c13e8 on 8.x-1.x
    Issue #2792897 by dpi: Coding standards
    
dpi’s picture

Status: Needs review » Fixed
dpi’s picture

Status: Fixed » Needs review
StatusFileSize
new24.49 KB

Short array syntax arrays

  • dpi committed d93f7d2 on 8.x-1.x
    Issue #2792897 by dpi: Convert arrays to short syntax.
    
dpi’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

dpi’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new36.17 KB

Status: Needs review » Needs work

The last submitted patch, 14: 2792897-cs.patch, failed testing. View results

dpi’s picture

StatusFileSize
new36.06 KB
dpi’s picture

Status: Needs work » Needs review

  • dpi committed bbe5360 on 8.x-1.x
    Issue #2792897 by dpi: Coding standards
    
dpi’s picture

Assigned: dpi » Unassigned
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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