- Issue created by @sunil jolapara
- Merge request !2Resolve #3476606 "Fix the phpcodesniffer issues" ā (Open) created by sunil jolapara
- šŗšø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.
- Assigned to cleavinjosh
- šµšPhilippines cleavinjosh
Hi @sunil jolapara,
Thank you for providing an MR. I applied MR!1 on 8.x-1.x, it was applied smoothly and fixed all of the phpcs issues.
ā access_conditions_field_group git:(8.x-1.x) curl https://git.drupalcode.org/project/access_conditions_field_group/-/merge_requests/2.diff | patch -p1 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 2516 0 2516 0 0 5831 0 --:--:-- --:--:-- --:--:-- 5824 patching file README.md patching file access_conditions_field_group.module patching file composer.json ā access_conditions_field_group git:(8.x-1.x) ā .. ā contrib git:(main) ā phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml access_conditions_field_group ā contrib git:(main) āThank you.
- š®š¹Italy apaderno Brescia, š®š¹
The merge request is still not using GitLab CI to get the list of the issues to fix.
- Merge request !3Created a new merge request to get the list of all the PHP_CodeSniffer errors/warnings to fix ā (Open) created by cleavinjosh
- šµšPhilippines cleavinjosh
Hi,
I created a new branch from 8.x-1.x to try to get the list of phpcs issues. However, there might have been some composer dependency resolution issues.
I am not sure how to proceed.
Please check and advise. Thank you.
- š®š¹Italy apaderno Brescia, š®š¹
The supported branch is 2.x. Any branch not compatible with Drupal 10 is no longer supported.