vendor/bin/phpcs --standard=Drupal --extensions=php,module,inc,install,profile,theme,yml web/modules/contrib/alertbox/

FILE: /home/sahana/Sites/contribution/web/modules/contrib/alertbox/alertbox.module
----------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
47 | ERROR | [ ] Doc comment short description must be on a single line, further text should be a separate paragraph
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------

FILE: /home/sahana/Sites/contribution/web/modules/contrib/alertbox/src/Form/AlertboxSettingsForm.php
----------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 7 WARNINGS AFFECTING 8 LINES
----------------------------------------------------------------------------------------------------
42 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters
80 | ERROR | [x] Whitespace found at end of line
106 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
107 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
108 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
122 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
123 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
124 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------

Time: 443ms; Memory: 10MB

CommentFileSizeAuthor
#3 alertboxclean.png263.76 KBguilhermevp
#2 3203217-1.patch2.58 KBsahana _n

Comments

Sahana _N created an issue. See original summary.

sahana _n’s picture

sahana _n commented
Status: Active » Needs review
StatusFileSize
new 3203217-1.patch2.58 KB

Please review the patch.

guilhermevp’s picture

guilhermevp commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new alertboxclean.png263.76 KB

Patch applies cleanly and phpcs returns no errors.

Image evidence attached.

  • rutiolma committed 87dc56d on 8.x-1.x authored by Sahana _N 
    Issue #3203217 by Sahana _N, guilhermevp: Phpcs coding standard
rutiolma’s picture

rutiolma
Location 🇵🇹
commented

Committed, thanks

rutiolma’s picture

rutiolma
Location 🇵🇹
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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