In CoffeeCommandsTest.php, the first multiline array doesn't have the trailing comma which Drupal coding standards require.

CommentFileSizeAuthor
#3 coffee-missing-comma-2923685-3-8.patch477 bytesAnonymous (not verified)
#2 2923685-2.patch498 bytesabacaba
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xdevx created an issue. See original summary.

abacaba’s picture

abacaba CreditAttribution: abacaba commented
FileSize
2923685-2.patch498 bytes

I've attached the patch

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous at Ashday Interactive Systems commented
Issue summary: View changes
FileSize
coffee-missing-comma-2923685-3-8.patch477 bytes

The patch did not quite apply for me, apparently due to the whitespace at the end of the patch file. Here is an updated patch to make the same change which works for me.

leopaccanaro’s picture

leopaccanaro CreditAttribution: leopaccanaro at CI&T commented
Status: Needs review » Reviewed & tested by the community

RTBC

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
CreditAttribution: joegraduate at The University of Arizona commented
Parent issue: » #3111053: Plan for Coffee 8.x-1.0 stable release

michaelmol’s picture

michaelmol CreditAttribution: michaelmol at Immer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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