- Issue created by @GGH
- Merge request !59Issue #3513897: Fix the issues reported by phpcs ā (Open) created by Unnamed author
- šµšPhilippines paraderojether
Hi
I reviewed MR!59 confirmed it passed phcps using Gitlab CI. However there are still phpcs issues found using PHP_CodeSniffer shown below:
ā maxlength git:(main) ā curl https://git.drupalcode.org/project/maxlength/-/merge_requests/59.diff | patch -p1 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 5215 0 5215 0 0 17373 0 --:--:-- --:--:-- --:--:-- 17677 patching file maxlength.install patching file maxlength.module patching file 'tests/src/FunctionalJavascript/MaxLengthCustomLinkWidgetTest.php' patching file 'tests/src/FunctionalJavascript/MaxLengthCustomWidgetTest.php' ā maxlength git:(main) ā cd .. ā contrib git:(main) ā phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig maxlength FILE: /Users/jetherparadero/Q1/q1issues/web/modules/contrib/maxlength/tests/src/FunctionalJavascript/MaxLengthCustomLinkWidgetTest.php -------------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE -------------------------------------------------------------------------------------------------------------------------------------- 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\field\Entity\FieldStorageConfig. -------------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------------------------------------------------------------------- FILE: /Users/jetherparadero/Q1/q1issues/web/modules/contrib/maxlength/maxlength.module ---------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ---------------------------------------------------------------------------------------------------------------------------- 11 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Field\WidgetInterface. ---------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ---------------------------------------------------------------------------------------------------------------------------- Time: 304ms; Memory: 12MB
Will retain the status to Needs Review so that others can also check.
Thank you.