Skip to content

Commit

Permalink
Merge pull request #58 from OggettoWeb/2.0.0_dev
Browse files Browse the repository at this point in the history
Remove required param comment
  • Loading branch information
dankocherga committed Oct 6, 2015
2 parents a3d53f9 + 778e911 commit 54978b1
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions Oggetto/Sniffs/Commenting/FunctionCommentSniff.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@
* <li>There is a blank newline between the long and short description.</li>
* <li>There is a blank newline between the long description and tags.</li>
* <li>Parameter names represent those in the method.</li>
* <li>Parameter comments are in the correct order</li>
* <li>Parameter comments are complete</li>
* <li>A space is present before the first and after the last parameter</li>
* <li>A return type exists</li>
* <li>There must be one blank line between body and headline comments.</li>
Expand Down Expand Up @@ -430,15 +428,6 @@ protected function processParams($commentStart)
$this->currentFile->addError($error, $errorPos, 'MissingParamType');
}

if ($paramComment === '') {
$error = 'Missing comment for param "%s" at position %s';
$data = array(
$paramName,
$pos,
);
$this->currentFile->addError($error, $errorPos, 'MissingParamComment', $data);
}

$previousParam = $param;

}//end foreach
Expand Down

0 comments on commit 54978b1

Please sign in to comment.