Problem/Motivation

I have scanned the module with PHPCS ../vendor/bin/phpcs --standard=drupal modules/contrib/pathauto
and found that there is a lot to do.

But there is already an issue Fix the issues reported by phpcs, which has been postponed because the patch was very heavy.

So I have created the issue according to the error.
It will help to review and merge easily.

Steps to reproduce

Proposed resolution

Scan the module using PHPCS and rearrange all use statements alphabetically.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork pathauto-3376723

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

keshav.k created an issue. See original summary.

keshav.k opened merge request !49

keshavv’s picture

keshavv commented
Status: Active » Needs review

I have created the MR please review it.
Thank you.

Berdir made their first commit to this issue’s fork.

  • Berdir committed 98bcd4a5 on 8.x-1.x authored by keshavv 
    Issue #3376723: Sort 'use' statements alphabetically
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

I'm always a bit torn on issues like this and whether it's worth spending your and my time on it.

But the new GitlabCI workflow benefits from fixing CS and it's verifiable that this fixes all of them and reduces the violation count.

Merged.

Status: Fixed » Closed (fixed)

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