The CoreStatusReport.php file needs to be updated for coding standards

CommentFileSizeAuthor
#3 admin_status-CoreStatusReport-code-standards-2883178-3.patch5.81 KBvolkswagenchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review

The CoreStatusReport.php file has been updated for coding standards. Patch uploaded.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
FileSize
admin_status-CoreStatusReport-code-standards-2883178-3.patch5.81 KB

ooops, forgot the patch. Here it is.

kristindev’s picture

kristindev CreditAttribution: kristindev at Hook 42 commented
Status: Needs review » Reviewed & tested by the community

This is sane and follows coding standards. Looks good!

loopduplicate credited ChrisDarke.

loopduplicate credited loopduplicate.

loopduplicate’s picture

loopduplicate
she/her
Location Burning Token, California
CreditAttribution: loopduplicate at Hook 42 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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