I saw that core's phpcs config file was recently updated. I applied those rules to our own config file and I saw that some other small things slipped in.

Not sure if I should've reopened #2671288: Follow drupal coding standards instead.

CommentFileSizeAuthor
#2 coding-standards-update.patch2.9 KBborisson_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

borisson_ created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and commented
Status: Active » Needs review
FileSize
coding-standards-update.patch2.9 KB
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Status: Needs review » Fixed

Not sure if I should've reopened #2671288: Follow drupal coding standards instead.

No, it's great this way, thanks!
And thanks a lot for the great job, once again! I'll just believe you that the phpcs.xml changes are correct.
So, committed.
Thanks again!

Status: Fixed » Closed (fixed)

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