Follow up to #2935151: Long array syntax checking in ./core files is broken

> phpcs --standard=core/phpcs.xml.dist --runtime-set installed_paths $($COMPOSER_BINARY config vendor-dir)/drupal/coder/coder_sniffer --

FILE: ...re/tests/Drupal/Tests/Listeners/DrupalStandardsListenerTrait.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 160 | ERROR | [x] Short array syntax must be used to define arrays
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 1 mins, 21.72 secs; Memory: 98Mb
CommentFileSizeAuthor
#2 2935524.patch705 bytesneclimdul
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

neclimdul created an issue. See original summary.

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Status: Active » Needs review
FileSize
2935524.patch705 bytes

At least we know the array check is working :)

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Needs review » Reviewed & tested by the community
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Reviewed & tested by the community » Fixed

Committed 1f0b760 and pushed to 8.5.x.

  • larowlan committed 1f0b760 on 8.5.x 
    Issue #2935524 by neclimdul: PHPCS failure on...

Status: Fixed » Closed (fixed)

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