Problem/Motivation

FILE: /home/lenovo/druapl7to9/web/modules/contrib/config_ignore_auto/config_ignore_auto.module
-----------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------------------
   1 | ERROR | [x] Missing file doc comment
  14 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 112 | ERROR | [x] Expected 1 blank line after function; 2 found
-----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------


FILE: /home/lenovo/druapl7to9/web/modules/contrib/config_ignore_auto/src/Form/Settings.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is
   |       |     Symfony\Component\DependencyInjection\ContainerInterface.
--------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------


FILE: /home/lenovo/druapl7to9/web/modules/contrib/config_ignore_auto/README.txt
-------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------
 30 | ERROR | [x] Expected 1 newline at end of file; 2 found
-------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------

Time: 81ms; Memory: 10MB

Steps to reproduce

Execute Command : phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig config_ignore_auto/

Proposed resolution

Remaining tasks

FILE: /home/lenovo/druapl7to9/web/modules/contrib/config_ignore_auto-3372903/config_ignore_auto.module
------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------
3 | ERROR | Missing short description in doc comment
------------------------------------------------------------------------------------------------------

Time: 73ms; Memory: 10MB

User interface changes

API changes

Data model changes

Issue fork config_ignore_auto-3372903

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

roshni27 created an issue. See original summary.

roshni27’s picture

roshni27 commented
Issue summary: View changes

roshni27 opened merge request !2

roshni27’s picture

roshni27 commented
Assigned: roshni27 » Unassigned
Status: Active » Needs review

I have solved phpcs issues except one . Please review MR .

silvi.addweb made their first commit to this issue’s fork.

clarkssquared’s picture

clarkssquared commented
Status: Needs review » Needs work

Hi

I applied the updated MR !2 and I can see that there's a PHPCS warning about Translatable String that the PHPCS reported

➜  contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml config_ignore_auto

FILE: ...9-local/web/modules/contrib/config_ignore_auto/config_ignore_auto.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
--------------------------------------------------------------------------------
 1 | WARNING | Remove "project" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "datestamp" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "version" from the info file, it will be added by
   |         | drupal.org packaging automatically
--------------------------------------------------------------------------------


FILE: ...s/d9/d9-local/web/modules/contrib/config_ignore_auto/src/Form/Settings.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 96 | WARNING | Translatable strings must not begin or end with white spaces,
    |         | use placeholders with t() for variables
--------------------------------------------------------------------------------

Time: 244ms; Memory: 10MB

➜  contrib git:(master) ✗
hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Needs work » Closed (won't fix)

I fixed this for 3.x, 2.x will likely receive no further update unless security issues arises.