Review of the 8.x-2.x branch (commit 515e04d):
- Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
FILE: ...eviewsh/pareview_temp/tests/src/Unit/Services/EntityServicesTest.php -------------------------------------------------------------------------- FOUND 7 ERRORS AND 4 WARNINGS AFFECTING 10 LINES -------------------------------------------------------------------------- 3 | ERROR | [x] Namespaced classes, interfaces and traits should not | | begin with a file doc comment 12 | ERROR | [x] Missing function doc comment 12 | ERROR | [x] Expected 1 blank line before function; 0 found 43 | ERROR | [x] Expected 1 blank line after function; 0 found 48 | ERROR | [ ] USE declarations must go after the first namespace | | declaration 49 | ERROR | [ ] USE declarations must go after the first namespace | | declaration 50 | ERROR | [ ] USE declarations must go after the first namespace | | declaration 273 | WARNING | [ ] Line exceeds 80 characters; contains 84 characters 317 | WARNING | [ ] Line exceeds 80 characters; contains 88 characters 341 | WARNING | [ ] Line exceeds 80 characters; contains 84 characters 385 | WARNING | [ ] Line exceeds 80 characters; contains 88 characters -------------------------------------------------------------------------- PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------- Time: 2.54 secs; Memory: 10Mb