Revision of http://git.drupal.org/project/blizz_dynamic_entitylist.git 8.x-2.x from Wed, 03/28/2018 - 19:50

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.

This automated report was generated with PAReview.sh, your friendly project application review script.

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