Fix all the issues reported by PHPCS for Drupal and DrupalPractice coding standards
Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,js,info,txt,md,yml,twig farm_asset_termination/
Fix all the issues and execute the phpcbf command to beautify the code.
No issues remaining to fix.
The patch provided fixes the following issues. Please review the patch:
FILE: .../web/modules/contrib/farm_asset_termination/farm_asset_termination.install
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
13 | WARNING | Line exceeds 80 characters; contains 88 characters
--------------------------------------------------------------------------------
FILE: ...contribution/drupal10/web/modules/contrib/farm_asset_termination/README.md
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
18 | WARNING | Line exceeds 80 characters; contains 87 characters
--------------------------------------------------------------------------------
FILE: ...web/modules/contrib/farm_asset_termination/farm_asset_termination.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
7 | WARNING | All dependencies must be prefixed with the project name, for
| | example "drupal:"
--------------------------------------------------------------------------------
FILE: .../contrib/farm_asset_termination/src/EventSubscriber/LogEventSubscriber.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
27 | ERROR | Missing @var tag in member variable comment
--------------------------------------------------------------------------------
FILE: ...b/modules/contrib/farm_asset_termination/src/AssetTerminationInterface.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------
30 | WARNING | Line exceeds 80 characters; contains 88 characters
47 | WARNING | Line exceeds 80 characters; contains 88 characters
--------------------------------------------------------------------------------
FILE: ...s/contrib/farm_asset_termination/src/Form/AssetTerminationSettingsForm.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
21 | ERROR | Missing @var tag in member variable comment
26 | ERROR | Missing @var tag in member variable comment
--------------------------------------------------------------------------------
FILE: ...upal10/web/modules/contrib/farm_asset_termination/src/AssetTermination.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
25 | ERROR | Missing @var tag in member variable comment
30 | ERROR | Missing @var tag in member variable comment
--------------------------------------------------------------------------------
FILE: ...0/web/modules/contrib/farm_asset_termination/farm_asset_termination.module
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
68 | WARNING | Line exceeds 80 characters; contains 94 characters
--------------------------------------------------------------------------------
Time: 715ms; Memory: 10MB
1.0
Code
It involves compliance with, or the content of coding standards. Requires broad community agreement.
Hi urvashi_vora
I applied the patch coding-standard-fixes.patch → to the “farmOS Asset Termination” module against Version 1.x-dev and confirmed that warnings are resolved. Please see the screenshots attached.
For your review.
Thank you.