Fix the PHP_CodeSniffer warnings/errors shown in GitLab CI tests

Fixed the issue from phpcs using command phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml. Please review.

@sakthi_dev Thanks for working on the issue.

I can still see some errors/warning on the phpcs pipeline build

can we address that ?

Thanks

sarwan_verma → made their first commit to this issue’s fork.

Hi @akshay_d,
I have fixed this issue "Fix the PHPCS errors reported for the project". created MR.
please review and verify.

Hi

I applied MR !7 and I confirmed that it fixes all the PHPCS issues and erros in the module

➜  style_entity git:(2.x) curl https://git.drupalcode.org/project/style_entity/-/merge_requests/7.diff | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5438    0  5438    0     0   9942      0 --:--:-- --:--:-- --:--:-- 10241
patching file 'src/Plugin/EntityReferenceSelection/UsesSelection.php'
patching file 'src/Plugin/Field/FieldType/StyleEntityReferenceItem.php'
patching file style_entity.module
patching file style_entity.post_update.php
patching file 'tests/src/Kernel/StyleEntityTest.php'
➜  style_entity git:(2.x) ✗ ..
➜  contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml style_entity 
➜  contrib git:(master) ✗ 

A summary must describe what needs to be changed, and eventually why. A screenshot is not sufficient, even in the case a patch/MR is provided.
In the case of bug reports, the steps to reproduce the bug should be provided too. In the case of coding standards issues found using an automatic tool, the report given by that tool must be quoted too, including the arguments passed to that tool, if it is a command line tool.

Also, please do not use a screenshot to show the report generated by PHP_CodeSniffer or other CLI tools.

Issue summary updated

There are still remaining issues that needs to get fixed. Updated the IS.

Providing the updated patch for 2.x

Please review.

Hi,

The patch #13 applied cleanly.
All the errors are fixed.
Please refer the screenshot.

Also, the MR changes five files, while the report is only for one file. Either the report is not complete, or the MR is changing more files than necessary.

I have applied !MR7 no phpcs found.
It looks good to me.