FILE: /opt/lampp/htdocs/contribution/modules/easy_breadcrumb/tests/src/Kernel/EasyBreadcrumbBuilderTest.php
-----------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------
 20 | ERROR | Missing member variable doc comment
-----------------------------------------------------------------------------------------------------------


FILE: /opt/lampp/htdocs/contribution/modules/easy_breadcrumb/easy_breadcrumb.install
------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
------------------------------------------------------------------------------------
 47 | WARNING | [ ] Line exceeds 80 characters; contains 91 characters
 69 | ERROR   | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------
CommentFileSizeAuthor
#2 3075090-coding-standard.patch1.2 KBShivalik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Shivalik created an issue. See original summary.

Shivalik’s picture

Shivalik CreditAttribution: Shivalik as a volunteer commented
Status: Active » Needs review
FileSize
3075090-coding-standard.patch1.2 KB

Errors Fixed.
Please review the patch.

TimeBandit’s picture

TimeBandit CreditAttribution: TimeBandit commented

Applying the patch works for me.

junkuncz’s picture

junkuncz CreditAttribution: junkuncz at Petend commented
Status: Needs review » Reviewed & tested by the community

Seems good to me!

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented

Looks good

  • RenatoG committed 949cb76 on 8.x-1.x authored by Shivalik 
    Issue #3075090 by Shivalik, TimeBandit, junkuncz, RenatoG: Coding...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Reviewed & tested by the community » Fixed

Committed to the dev branch.

Thank you all

Status: Fixed » Closed (fixed)

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