Problem/Motivation

FILE: /var/www/html/vb/d_cont/link_selection_handler/README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------
3 | WARNING | Line exceeds 80 characters; contains 164 characters
4 | WARNING | Line exceeds 80 characters; contains 144 characters
6 | WARNING | Line exceeds 80 characters; contains 99 characters
----------------------------------------------------------------------

FILE: ...ar/www/html/vb/d_cont/link_selection_handler/link_selection_handler.module
--------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------
4 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
5 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
6 | ERROR | [x] Expected 1 space(s) before asterisk; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 93ms; Memory: 10MB

Steps to reproduce

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

Remaining tasks

Patch review

CommentFileSizeAuthor
coding-standard-fixes.patch2.32 KBurvashi_vora

Comments

urvashi_vora created an issue. See original summary.

frederikvho’s picture

frederikvho
him/he
Primary language Dutch
Location Belgium
commented
Status: Needs review » Reviewed & tested by the community

Hi,

I applied the patch from the issue summary and ran PHPCS again. It threw no more warnings/errors. So this looks good.

 (base) ➜  link_selection_handler git:(1.0.x) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig .
(base) ➜  link_selection_handler git:(1.0.x) ✗

  • mradcliffe committed a79d9987 on 1.0.x 
    Issue #3373227 by urashi_vora, Frederikvho, mradcliffe: Fixes issues...
mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thank you for the patch. I reviewed it, and applied the .module fixes.

Status: Fixed » Closed (fixed)

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