There are still files other than README.txt in the master branch, make sure to remove them. See also step 5 in http://drupal.org/node/1127732 The following git branches do not match the release branch pattern, you should remove/rename them. See http://drupal.org/node/1015226 remotes/origin/7.x-1.0 Review of the 7.x-1.x branch: This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me. Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards sites/all/modules/pareview_temp/./test_candidate/includes/hooks.inc: +45: [normal] Menu item titles and descriptions should NOT be enclosed within t(). +54: [normal] Menu item titles and descriptions should NOT be enclosed within t(). +62: [normal] Menu item titles and descriptions should NOT be enclosed within t(). Status Messages: Coder found 8 projects, 8 files, 3 normal warnings, 0 warnings were flagged to be ignored FILE: ...al-7-pareview/sites/all/modules/pareview_temp/test_candidate/README.txt -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 24 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...review/sites/all/modules/pareview_temp/test_candidate/contact_book.info -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 5 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...iew/sites/all/modules/pareview_temp/test_candidate/contact_book.install -------------------------------------------------------------------------------- FOUND 7 ERROR(S) AFFECTING 7 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 3 | ERROR | Missing function doc comment 9 | ERROR | Missing function doc comment 52 | ERROR | Missing function doc comment 65 | ERROR | Missing function doc comment 71 | ERROR | Missing function doc comment 75 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...view/sites/all/modules/pareview_temp/test_candidate/contact_book.module -------------------------------------------------------------------------------- FOUND 6 ERROR(S) AFFECTING 6 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 5 | ERROR | Missing function doc comment 24 | ERROR | Missing function doc comment 36 | ERROR | Missing function doc comment 43 | ERROR | else must start on a new line 46 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...iew/sites/all/modules/pareview_temp/test_candidate/css/contact_book.css -------------------------------------------------------------------------------- FOUND 10 ERROR(S) AFFECTING 6 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Expected 1 space after colon in style definition; 0 found 5 | ERROR | Expected 1 space before opening brace of class definition; 0 | | found 5 | ERROR | Expected exactly one new line after opening brace of class | | definition 5 | ERROR | Class definition is empty 7 | ERROR | Expected exactly one new line before closing brace of class | | definition 8 | ERROR | Expected 1 space before opening brace of class definition; 0 | | found 8 | ERROR | Expected exactly one new line after opening brace of class | | definition 8 | ERROR | Class definition is empty 10 | ERROR | Expected exactly one new line before closing brace of class | | definition 31 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/includes/admin.inc -------------------------------------------------------------------------------- FOUND 6 ERROR(S) AFFECTING 6 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 3 | ERROR | Missing function doc comment 6 | ERROR | Array indentation error, expected 4 spaces but found 6 7 | ERROR | Array indentation error, expected 4 spaces but found 6 8 | ERROR | Array indentation error, expected 4 spaces but found 6 12 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...review/sites/all/modules/pareview_temp/test_candidate/includes/ajax.inc -------------------------------------------------------------------------------- FOUND 6 ERROR(S) AFFECTING 6 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 12 | ERROR | else must start on a new line 26 | ERROR | else must start on a new line 53 | ERROR | else must start on a new line 61 | ERROR | else must start on a new line 64 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/includes/forms.inc -------------------------------------------------------------------------------- FOUND 57 ERROR(S) AND 1 WARNING(S) AFFECTING 56 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 24 | ERROR | Array indentation error, expected 4 spaces but found 6 25 | ERROR | Array indentation error, expected 4 spaces but found 6 26 | ERROR | Array indentation error, expected 4 spaces but found 6 27 | ERROR | Array indentation error, expected 4 spaces but found 6 31 | ERROR | Array indentation error, expected 4 spaces but found 6 34 | ERROR | Array indentation error, expected 4 spaces but found 6 35 | ERROR | Array indentation error, expected 4 spaces but found 6 36 | ERROR | Array indentation error, expected 4 spaces but found 6 37 | ERROR | Array indentation error, expected 4 spaces but found 6 40 | ERROR | Array indentation error, expected 4 spaces but found 6 41 | ERROR | Array indentation error, expected 4 spaces but found 6 42 | ERROR | Array indentation error, expected 4 spaces but found 6 43 | ERROR | Array indentation error, expected 4 spaces but found 6 44 | ERROR | Array indentation error, expected 4 spaces but found 6 47 | ERROR | Array indentation error, expected 4 spaces but found 6 48 | ERROR | Array indentation error, expected 4 spaces but found 6 49 | ERROR | Array indentation error, expected 4 spaces but found 6 50 | ERROR | Array indentation error, expected 4 spaces but found 6 53 | ERROR | Array indentation error, expected 4 spaces but found 6 54 | ERROR | Array indentation error, expected 4 spaces but found 6 55 | ERROR | Array indentation error, expected 4 spaces but found 6 56 | ERROR | Array indentation error, expected 4 spaces but found 6 63 | ERROR | Array indentation error, expected 4 spaces but found 6 64 | ERROR | Array indentation error, expected 4 spaces but found 6 65 | ERROR | Array indentation error, expected 4 spaces but found 6 69 | ERROR | Array indentation error, expected 4 spaces but found 6 70 | ERROR | Array indentation error, expected 4 spaces but found 6 71 | ERROR | Array indentation error, expected 4 spaces but found 6 72 | ERROR | Array indentation error, expected 4 spaces but found 6 73 | ERROR | Array indentation error, expected 4 spaces but found 6 73 | ERROR | TRUE, FALSE and NULL must be uppercase; expected "FALSE" but | | found "false" 77 | ERROR | Array indentation error, expected 4 spaces but found 6 78 | ERROR | Array indentation error, expected 4 spaces but found 6 79 | ERROR | Array indentation error, expected 4 spaces but found 6 83 | ERROR | Array indentation error, expected 4 spaces but found 6 84 | ERROR | Array indentation error, expected 4 spaces but found 6 85 | ERROR | Array indentation error, expected 4 spaces but found 6 92 | ERROR | Function comment short description must start with a capital | | letter 103 | ERROR | Function comment short description must start with a capital | | letter 112 | ERROR | else must start on a new line 121 | ERROR | Array indentation error, expected 4 spaces but found 6 131 | ERROR | Inline comments must end in full-stops, exclamation marks, or | | question marks 152 | ERROR | Array indentation error, expected 4 spaces but found 6 153 | ERROR | Array indentation error, expected 4 spaces but found 6 154 | ERROR | Array indentation error, expected 4 spaces but found 6 155 | ERROR | Array indentation error, expected 4 spaces but found 6 156 | ERROR | Array indentation error, expected 8 spaces but found 10 157 | ERROR | Array indentation error, expected 8 spaces but found 10 158 | ERROR | Array indentation error, expected 8 spaces but found 10 159 | ERROR | Array indentation error, expected 4 spaces but found 6 166 | WARNING | Line exceeds 80 characters; contains 85 characters 166 | ERROR | Comments may not appear after statements. 173 | ERROR | Array indentation error, expected 4 spaces but found 6 174 | ERROR | Array indentation error, expected 4 spaces but found 6 175 | ERROR | Array indentation error, expected 4 spaces but found 6 176 | ERROR | Array indentation error, expected 4 spaces but found 6 187 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/includes/hooks.inc -------------------------------------------------------------------------------- FOUND 31 ERROR(S) AND 5 WARNING(S) AFFECTING 36 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 4 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 27 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 31 | ERROR | Array indentation error, expected 4 spaces but found 6 32 | ERROR | Array indentation error, expected 8 spaces but found 10 33 | ERROR | Array indentation error, expected 4 spaces but found 6 38 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 45 | ERROR | Array indentation error, expected 4 spaces but found 6 46 | ERROR | Array indentation error, expected 4 spaces but found 6 47 | ERROR | Array indentation error, expected 4 spaces but found 6 48 | ERROR | Array indentation error, expected 4 spaces but found 6 49 | ERROR | Array indentation error, expected 4 spaces but found 6 50 | ERROR | Array indentation error, expected 4 spaces but found 6 54 | ERROR | Array indentation error, expected 4 spaces but found 6 55 | ERROR | Array indentation error, expected 4 spaces but found 6 56 | ERROR | Array indentation error, expected 4 spaces but found 6 57 | ERROR | Array indentation error, expected 4 spaces but found 6 58 | ERROR | Array indentation error, expected 4 spaces but found 6 62 | ERROR | Array indentation error, expected 4 spaces but found 6 63 | ERROR | Array indentation error, expected 4 spaces but found 6 64 | ERROR | Array indentation error, expected 4 spaces but found 6 65 | ERROR | Array indentation error, expected 4 spaces but found 6 69 | ERROR | Array indentation error, expected 4 spaces but found 6 70 | ERROR | Array indentation error, expected 4 spaces but found 6 71 | ERROR | Array indentation error, expected 4 spaces but found 6 72 | ERROR | Array indentation error, expected 4 spaces but found 6 76 | ERROR | Array indentation error, expected 4 spaces but found 6 77 | ERROR | Array indentation error, expected 4 spaces but found 6 78 | ERROR | Array indentation error, expected 4 spaces but found 6 79 | ERROR | Array indentation error, expected 4 spaces but found 6 86 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 92 | ERROR | Array indentation error, expected 4 spaces but found 6 93 | ERROR | Array indentation error, expected 4 spaces but found 6 94 | ERROR | Array indentation error, expected 4 spaces but found 6 101 | WARNING | Format should be "* Implements hook_foo()." or "Implements | | hook_foo_BAR_ID_bar() for xyz_bar()." 108 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/includes/theme.inc -------------------------------------------------------------------------------- FOUND 4 ERROR(S) AFFECTING 4 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 3 | ERROR | Missing function doc comment 11 | ERROR | else must start on a new line 17 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...review/sites/all/modules/pareview_temp/test_candidate/includes/util.inc -------------------------------------------------------------------------------- FOUND 9 ERROR(S) AFFECTING 7 LINE(S) -------------------------------------------------------------------------------- 2 | ERROR | Missing file doc comment 3 | ERROR | Missing function doc comment 4 | ERROR | If the line declaring an array spans longer than 80 characters, | | each element should be broken into its own line 20 | ERROR | If the line declaring an array spans longer than 80 characters, | | each element should be broken into its own line 24 | ERROR | The first index in a multi-value array must be on a new line 24 | ERROR | The first index in a multi-value array must be on a new line 24 | ERROR | The first index in a multi-value array must be on a new line 34 | ERROR | Missing function doc comment 40 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/js/contact_book.js -------------------------------------------------------------------------------- FOUND 1 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 20 | ERROR | Files must end in a single new line character -------------------------------------------------------------------------------- FILE: ...ules/pareview_temp/test_candidate/templates/contact-book-output.tpl.php -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 1 LINE(S) -------------------------------------------------------------------------------- 1 | ERROR | Missing file doc comment 1 | ERROR | Files must end in a single new line character --------------------------------------------------------------------------------
Source: http://ventral.org/pareview - PAReview.sh online service