Re: QA: Package coding standards check
| From: | Alan Knowles | Date: | Mon, 22 Oct 2007 23:09:57 +0000 |
| Subject: | Re: QA: Package coding standards check | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-48298@lists.php.net to get a copy of this message | ||
Christian Weiske wrote:
Hello all you pear developers, Our quality assurance suite now utilizes our great package PHP_CodeSniffer to generate reports about each package's coding standards compliance. You can get the full overview at http://pear.cweiske.de/overview.htm Just click at the CS error/warning count to get a detailled listing of problems in the package. I'm sure there are quite a few bugs on it ;)Things that look like spurious errors. - @***** tag comment indented incorrectly. Expected 3 spaces but found 4. - Line indented incorrectly; expected 8 spaces, found 12 (related to the case issue with the CS) While it's a CS issue, some issues are more serious than others? - No space found after comma in function call I'm sure there are other issues, but it would help to see the line that the Error is talking about displayed next to or after the error Regards Alan
Please also note that we're currently evaluating to extend and clarify the standards. See my last email to this list or http://wiki.pear.php.net/index.php/CodingStandardsChanges As always, comments are very welcome.