$ phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml simplenews_decoupled/
Xdebug: [Step Debug] Time-out connecting to debugging client, waited: 200 ms. Tried: localhost:9000 (through xdebug.client_host/xdebug.client_port) :-(

FILE: C:\Users\SI-001\Downloads\simplenews_decoupled\README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
----------------------------------------------------------------------
18 | WARNING | Line exceeds 80 characters; contains 88 characters
19 | WARNING | Line exceeds 80 characters; contains 99 characters
84 | WARNING | Line exceeds 80 characters; contains 117 characters
90 | WARNING | Line exceeds 80 characters; contains 111 characters
107 | WARNING | Line exceeds 80 characters; contains 84 characters
----------------------------------------------------------------------

Comments

Rakhi Soni created an issue. See original summary.

rakhi soni’s picture

Assigned: rakhi soni » Unassigned
Status: Active » Needs review
StatusFileSize
new2.22 KB

Kindly review patch,,

sourabhjain’s picture

Assigned: Unassigned » sourabhjain

I will review it.

sourabhjain’s picture

Assigned: sourabhjain » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new1.18 MB
new994.81 KB

I have reviewed the patch #2 and its looks good and its resolved all readme related issues. Attaching before and after Screenshot for reference.
Moving to RTBC.

  • daveiano committed d935b64 on 1.0.x authored by Rakhi Soni
    Issue #3302339 by Rakhi Soni, sourabhjain: Line exceeds 80 characters in...
daveiano’s picture

Version: 1.1.x-dev » 1.0.x-dev
Status: Reviewed & tested by the community » Fixed

Committed, thank @all!

Status: Fixed » Closed (fixed)

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