Skip to main content
PAReview
Toggle navigation
Home
All reviews
Review a Drupal project online
https://git.drupal.org/project/notify_to_slack.git 8.x-2.x
Primary tabs
View
Source
(active tab)
Repeat review
HTML Source
Review of the 8.x-2.x branch (commit 71059a6): <ul> <li>The notify_to_slack.module does not implement <code>hook_help()</code>. See https://www.drupal.org/docs/develop/documenting-your-project/module-documentation-guidelines#hook_help .</li> <li><a href="https://www.drupal.org/project/coder">Coder Sniffer</a> has found some issues with your code (please check the <a href="https://www.drupal.org/node/318">Drupal coding standards</a>). <code> FILE: ...000000/site1101/web/vendor/drupal/pareviewsh/pareview_temp/README.md -------------------------------------------------------------------------- FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES -------------------------------------------------------------------------- 14 | WARNING | Line exceeds 80 characters; contains 106 characters 32 | WARNING | Line exceeds 80 characters; contains 92 characters 40 | WARNING | Line exceeds 80 characters; contains 137 characters 41 | WARNING | Line exceeds 80 characters; contains 131 characters -------------------------------------------------------------------------- Time: 1.83 secs; Memory: 6Mb </code></li> <li>No automated test cases were found, did you consider writing <a href="https://www.drupal.org/phpunit">PHPUnit tests</a>? This is not a requirement but encouraged for professional software development.</li> </ul> <i>This automated report was generated with <a href="https://www.drupal.org/project/pareviewsh">PAReview.sh</a>, your friendly project application review script.</i>