Review of the 8.x-2.x branch (commit 71059a6):
- The notify_to_slack.module does not implement
hook_help()
. See https://www.drupal.org/docs/develop/documenting-your-project/module-docu... . - Coder Sniffer has found some issues with your code (please check the Drupal coding standards).
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
- No automated test cases were found, did you consider writing PHPUnit tests? This is not a requirement but encouraged for professional software development.