- Issue created by @sunil jolapara
- Status changed to Needs review
2 months ago 10:01am 20 September 2024 - šŗšøUnited States sunil jolapara 3265 Summitrun Drive, Independence, KY, 41051
- š®š¹Italy apaderno Brescia, š®š¹
The issue summary for PHP_CodeSniffer issues should always show the report given from PHP_CodeSniffer and the arguments passed to
phpcs
.Furthermore, since GitLab CI is now enabled for all the projects, GitLab CI should be used to get PHP_CodeSniffer errors/warnings.
- Merge request !2Created a new merge request to get the list of all the PHP_CodeSniffer errors/warnings to fix ā (Open) created by apaderno
- š®š¹Italy apaderno Brescia, š®š¹
avpaderno ā changed the visibility of the branch 3475741-gitlab-ci-reports to hidden.
- First commit to issue fork.
- šµšPhilippines cleavinjosh
cleavinjosh ā changed the visibility of the branch 3475741-phpcs-issues to hidden.
- šµšPhilippines cleavinjosh
Found more issues after I ran
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml
:ā contrib git:(main) ā phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml drutopia_people FILE: /Users/interns/Demo-site/drupal_11_test/install-dir/drupal_11_test_site/web/modules/contrib/drutopia_people/drutopia_people.info.yml ------------------------------------------------------------------------------------------------------------------------------------------ FOUND 0 ERRORS AND 21 WARNINGS AFFECTING 21 LINES ------------------------------------------------------------------------------------------------------------------------------------------ 6 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 7 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 8 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 9 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 10 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 11 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 12 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 13 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 14 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 15 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 16 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 17 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 18 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 19 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 20 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 21 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 22 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 23 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 24 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 25 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" 26 | WARNING | All dependencies must be prefixed with the project name, for example "drupal:" ------------------------------------------------------------------------------------------------------------------------------------------ Time: 166ms; Memory: 10MB ā contrib git:(main) ā
- šµšPhilippines cleavinjosh
Hi,
Please review, check, and advise on MR!1.
Thank you.