Having reviewed a module recently within the Security Advisory Coverage application queue, a discussion came up of whether hook_help() must be used or should be used to comply with the module documentation guidelines.
The guidelines state
All but the most trivial modules should implement hook_help().
in bold letters.
As we are currently also working on getting the PAReview service back online and fixing some issues with the corresponding script, I'm more than curious about the consensus of whether it is a MUST or a STRONG SUGGESTION. So we could mark it as "Warning" or "Error" accordingly when spreading the word and trying to have uniform modules.
Maybe adding an according explanation or changing the wording within the documentation guidelines would be the next step.