FILE: /root/repos/pareviewsh/pareview_temp/kashmir.drush.inc
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
332 | ERROR | [x] Expected 1 newline at end of file; 2 found
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
FILE: /root/repos/pareviewsh/pareview_temp/README.md
----------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------
7 | WARNING | Line exceeds 80 characters; contains 81 characters
9 | WARNING | Line exceeds 80 characters; contains 91 characters
11 | WARNING | Line exceeds 80 characters; contains 81 characters
48 | WARNING | Line exceeds 80 characters; contains 91 characters
----------------------------------------------------------------------
FILE: ...eviewsh/pareview_temp/components/_twig-components/tags/trans.tag.php
--------------------------------------------------------------------------
FOUND 13 ERRORS AFFECTING 12 LINES
--------------------------------------------------------------------------
18 | ERROR | [ ] Class name must use UpperCamel naming without
| | underscores
24 | ERROR | [x] Short array syntax must be used to define arrays
29 | ERROR | [x] Short array syntax must be used to define arrays
61 | ERROR | [x] Short array syntax must be used to define arrays
61 | ERROR | [x] Short array syntax must be used to define arrays
64 | ERROR | [x] Short array syntax must be used to define arrays
101 | ERROR | [x] Short array syntax must be used to define arrays
140 | ERROR | [x] Short array syntax must be used to define arrays
142 | ERROR | [x] Short array syntax must be used to define arrays
158 | ERROR | [ ] Class name must use UpperCamel naming without
| | underscores
180 | ERROR | [x] Short array syntax must be used to define arrays
184 | ERROR | [x] Short array syntax must be used to define arrays
201 | ERROR | [x] Short array syntax must be used to define arrays
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 11 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------
ESLint has found some issues with your code (please check the JavaScript coding standards).
/root/repos/pareviewsh/pareview_temp/components/_annotations/annotations.js: line 2, col 14, Error - Parsing error: Unexpected token :
Codespell has found some spelling errors in your code.
./kashmir.drush.inc:22: ommited ==> omitted
DrupalPractice has found some issues with your code, but could be false positives.
FILE: ...eviewsh/pareview_temp/components/_twig-components/tags/trans.tag.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------
18 | WARNING | Class name must be prefixed with the project name
| | "Kashmir.info"
158 | WARNING | Class name must be prefixed with the project name
| | "Kashmir.info
Remove LICENSE.txt, it will be added by drupal.org packaging automatically.
Git default branch is not set, see the documentation on setting a default branch.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2903794-4.patch | 2.99 KB | bhanuprakashnani |
| #2 | code_style_issues-2903794-2.patch | 5.42 KB | dhruveshdtripathi |
Comments
Comment #2
dhruveshdtripathi commentedComment #3
florisg commentedCleanup breaks working code, please refactor.
gulp patternlab --> php if error (in php 7.0)
Comment #4
bhanuprakashnani commentedMade the patch with the required syntax for arrays. Please review and mention if any more changes are to be made. Thank you.