Review of the 8.x-1.x branch (commit 801a0f8):
- Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
- No automated test cases were found, did you consider writing PHPUnit tests? This is not a requirement but encouraged for professional software development.
FILE: ...upal/pareviewsh/pareview_temp/src/Utils/EntityHttpExceptionUtils.php -------------------------------------------------------------------------- FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES -------------------------------------------------------------------------- 20 | WARNING | [x] A comma should follow the last multiline array item. | | Found: 'node' 24 | WARNING | [x] A comma should follow the last multiline array item. | | Found: 'taxonomy_term' -------------------------------------------------------------------------- PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------- FILE: .../vendor/drupal/pareviewsh/pareview_temp/entity_http_exception.module -------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE -------------------------------------------------------------------------- 1 | ERROR | [x] Missing file doc comment -------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------- Time: 465ms; Memory: 4Mb