$ phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml glossify/

FILE: C:\xampp\htdocs\test\web\modules\contrib\glossify\README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------
13 | WARNING | Line exceeds 80 characters; contains 185 characters
20 | WARNING | Line exceeds 80 characters; contains 84 characters
28 | WARNING | Line exceeds 80 characters; contains 84 characters
29 | WARNING | Line exceeds 80 characters; contains 108 characters
51 | WARNING | Line exceeds 80 characters; contains 94 characters
----------------------------------------------------------------------

FILE: ...docs\test\web\modules\contrib\glossify\tests\src\Unit\GlossifyBaseTest.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
160 | ERROR | [x] Array indentation error, expected 8 spaces but found 7
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 408ms; Memory: 10MB

CommentFileSizeAuthor
#4 after_patch.png101.25 KBsourabhjain
#2 drupal_coding_standard_issue-3292814-2.patch2.78 KBrakhi soni

Comments

Rakhi Soni created an issue. See original summary.

rakhi soni’s picture

rakhi soni commented
Assigned: rakhi soni » Unassigned
Status: Active » Needs review
StatusFileSize
new drupal_coding_standard_issue-3292814-2.patch2.78 KB

I have created a patch to fix this issue,, please review it.

Thanks & Regards
Rakhi Soni,,

sourabhjain’s picture

sourabhjain
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

I have reviewed it and the patch #2 looks fine.
Moving to RTBC

sourabhjain’s picture

sourabhjain
Primary language English
commented
StatusFileSize
new after_patch.png101.25 KB
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Reviewed & tested by the community » Fixed 
drupal_coding_standard_issue-3292814-2.patch:32: trailing whitespace.
  indicate whether or not every match should be linked or just the first 
drupal_coding_standard_issue-3292814-2.patch:33: trailing whitespace.
  occurrence. display the text from a selected field on the linked node as a 
warning: 2 lines add whitespace errors.

Fixed the above warnings and committed, thanks.

  • sanduhrs committed 7fc1c46 on 2.0.x authored by Rakhi Soni 
    Issue #3292814 by Rakhi Soni, sourabhjain: Drupal Coding Standard Issue...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.