Reviewed your module and found these errors - see attached file for details:

FILE: ...iew/sites/all/modules/pareview_temp/test_candidate/hide_modules.install

2 | ERROR | Missing file doc comment
2 | ERROR | Missing function doc comment
6 | ERROR | Line indented incorrectly; expected 2 spaces, found 0
6 | ERROR | 2 spaces found before inline comment; expected "//
| | variable_del('hide_modules_groups');" but found "//
| | variable_del('hide_modules_groups');"
7 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: ...view/sites/all/modules/pareview_temp/test_candidate/hide_modules.module

2 | ERROR | You must use "/**" style comments for a file comment
11 | ERROR | You must use "/**" style comments for a function comment
12 | ERROR | Inline control structures are not allowed
15 | ERROR | Inline control structures are not allowed
19 | WARNING | Line exceeds 80 characters; contains 92 characters
19 | ERROR | Line indented incorrectly; expected 2 spaces, found 0
19 | ERROR | 2 spaces found before inline comment; expected "// $hide_groups
| | = explode(',', variable_get('hide_modules_groups',
| | DEFAULT_MODULE_GROUPS));" but found "// $hide_groups =
| | explode(',', variable_get('hide_modules_groups',
| | DEFAULT_MODULE_GROUPS));"
19 | ERROR | There must be no blank line following an inline comment
21 | ERROR | Inline control structures are not allowed
25 | ERROR | You must use "/**" style comments for a function comment
28 | ERROR | Inline control structures are not allowed
32 | ERROR | Inline control structures are not allowed
39 | ERROR | You must use "/**" style comments for a function comment
50 | ERROR | Missing function doc comment
74 | ERROR | Line indented incorrectly; expected 2 spaces, found 0
74 | ERROR | 4 spaces found before inline comment; expected "//
| | 'hide_modules_groups' => array(" but found "//
| | 'hide_modules_groups' => array("

CommentFileSizeAuthor
ModulesError.txt3.45 KBdev001

Comments

amontero’s picture

Status: Needs work » Needs review

Thanks for your feedback, loginradius.
Getting ready for promotion to full project.
PAReview is giving me:
16 | ERROR | Missing function doc comment
35 | ERROR | Missing function doc comment
49 | ERROR | Missing function doc comment
74 | ERROR | Missing function doc comment
But those functions already have function doc comment. I've searched on d.o and some issues regarding false positives pop out. So, I'm assuming it's a false positive.
PAReview link: http://ventral.org/pareview/httpgitdrupalorgsandboxamontero1552392git-7x-2x

Further feedback is much appreciated.
TIA.