Problem/Motivation

We don't need to override DisallowLongArraySyntaxSnif anymore for Drupal 7 support. Drupal 7 users can still disable this sniff in their phpcs.xml config.

Proposed resolution

Remove the sniff and use the upstream Generic.Arrays.DisallowLongArraySyntax in our ruleset.

Remaining tasks

Finish pull request https://github.com/pfrenssen/coder/pull/275

API changes

Drupal.Arrays.DisallowLongArraySyntax is removed and replaced with Generic.Arrays.DisallowLongArraySyntax.

Data model changes

Comments

klausi created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Issue summary: View changes
Parent issue: » #3552621: Release new Coder 9 major upgrade

  • klausi authored 0dbc395c on 9.x 
    refactor(DisallowLongArraySyntax): Use upstream sniff directly (#3554286...
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Fixed

Merged.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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