Review of the 8.x-1.x branch (commit 324e0ee):
Your README.md does not follow best practices (headings need to be uppercase). See https://www.drupal.org/node/2181737 .
The INTRODUCTION section is missing.
The REQUIREMENTS section is missing.
The INSTALLATION section is missing.
The CONFIGURATION section is missing.

FILE: ...000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/README.md
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 33 WARNINGS AFFECTING 33 LINES
--------------------------------------------------------------------------
6 | WARNING | Line exceeds 80 characters; contains 209 characters
12 | WARNING | Line exceeds 80 characters; contains 123 characters
14 | WARNING | Line exceeds 80 characters; contains 87 characters
15 | WARNING | Line exceeds 80 characters; contains 86 characters
16 | WARNING | Line exceeds 80 characters; contains 88 characters
22 | WARNING | Line exceeds 80 characters; contains 101 characters
23 | WARNING | Line exceeds 80 characters; contains 98 characters
29 | WARNING | Line exceeds 80 characters; contains 84 characters
31 | WARNING | Line exceeds 80 characters; contains 93 characters
38 | WARNING | Line exceeds 80 characters; contains 82 characters
51 | WARNING | Line exceeds 80 characters; contains 107 characters
53 | WARNING | Line exceeds 80 characters; contains 85 characters
61 | WARNING | Line exceeds 80 characters; contains 185 characters
63 | WARNING | Line exceeds 80 characters; contains 125 characters
66 | WARNING | Line exceeds 80 characters; contains 115 characters
68 | WARNING | Line exceeds 80 characters; contains 162 characters
98 | WARNING | Line exceeds 80 characters; contains 112 characters
99 | WARNING | Line exceeds 80 characters; contains 113 characters
102 | WARNING | Line exceeds 80 characters; contains 146 characters
104 | WARNING | Line exceeds 80 characters; contains 94 characters
109 | WARNING | Line exceeds 80 characters; contains 113 characters
115 | WARNING | Line exceeds 80 characters; contains 117 characters
116 | WARNING | Line exceeds 80 characters; contains 142 characters
117 | WARNING | Line exceeds 80 characters; contains 242 characters
119 | WARNING | Line exceeds 80 characters; contains 96 characters
128 | WARNING | Line exceeds 80 characters; contains 93 characters
144 | WARNING | Line exceeds 80 characters; contains 108 characters
145 | WARNING | Line exceeds 80 characters; contains 102 characters
167 | WARNING | Line exceeds 80 characters; contains 88 characters
170 | WARNING | Line exceeds 80 characters; contains 91 characters
178 | WARNING | Line exceeds 80 characters; contains 141 characters
185 | WARNING | Line exceeds 80 characters; contains 170 characters
191 | WARNING | Line exceeds 80 characters; contains 105 characters

CommentFileSizeAuthor
#3 3109709-3.patch9.51 KBAnkush_03
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ankush_03’s picture

Ankush_03’s picture

Status: Active » Needs review
FileSize
9.51 KB

Adding patch !

Ankush_03’s picture

Ignoring phpcs line length issue in case of urls.

lussoluca’s picture

Status: Needs review » Fixed

Committed and pushed to 2.x branch
Thanks!

Status: Fixed » Closed (fixed)

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