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.

Comments

shalimanov created an issue. See original summary.

dhruveshdtripathi’s picture

Status: Active » Needs review
StatusFileSize
new5.42 KB
florisg’s picture

Status: Needs review » Needs work

Cleanup breaks working code, please refactor.
gulp patternlab --> php if error (in php 7.0)

bhanuprakashnani’s picture

Assigned: sstanislav » bhanuprakashnani
Status: Needs work » Needs review
StatusFileSize
new2.99 KB

Made the patch with the required syntax for arrays. Please review and mention if any more changes are to be made. Thank you.