PHPCS: too much noise?

From: Date: Tue, 11 Nov 2008 08:17:06 +0000
Subject: PHPCS: too much noise?
Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-51040@lists.php.net to get a copy of this message
Hi, I just managed to receive a first "bug" for HTTP_Request2 package, http://pear.php.net/bugs/bug.php?id=14990 dealing with supposed "Coding standards violations": http://pear.cweiske.de/codingstandards/HTTP_Request2.html I dutifully looked through the results and couldn't help noticing that 95% of stuff there is petty complaints dealing with PHPDoc comments: I'm quoting from the "errors" of HTTP_Request2 class: error 36 @category tag comment indented incorrectly. Expected 1 spaces but found 3. error 37 @package tag comment indented incorrectly. Expected 2 spaces but found 4. --- last time I checked, we DON'T have any coding standards that are dealing with indenting the stuff within docblocks. error 65 Line indented incorrectly; expected at least 4 spaces, found 3 error 78 Line indented incorrectly; expected at least 4 spaces, found 3 error 80 Line indented incorrectly; expected at least 4 spaces, found 3 --- this deals with the first line of doc comment blocks having an indent of 3 spaces, to align the asterisks with actual thing being documented: /**
    * Request URL
    * @var  Net_URL2
    */
    protected $url;
error 173 Expected 1 space before variable type error 173 Doc comment var "Request" does not match actual variable name "$url" at position 1 error 173 Expected 1 space after the longest type --- this once again deals with indenting the stuff within PHPDoc comments (remember, we DON'T have any standards for this) and with the fact that phpDocumentor is perfectly capable of parsing the line
    * @param    string|Net_Url2     Request URL
and building proper docs without me repeating the parameter name:
    * @param    string|Net_Url2     $url 	Request URL
And this BS goes on and on and on... Now a question: how is one supposed to find actual coding standards violations in this steaming pile? Another question: if doc comments are mostly intended for phpDocumentor, which is perfectly capable of building proper docs without caring about such "errors", what's the point?

« previous php.pear.dev (#51040) next »