Re: Re: (Christian Weiske) PEAR QA tools page
| From: | Jorrit Schippers - nCode | Date: | Sun, 16 Dec 2007 10:54:18 +0000 |
| Subject: | Re: Re: (Christian Weiske) PEAR QA tools page | ||
| References: | 1 2 3 | Groups: | php.pear.qa |
| Request: | Send a blank email to pear-qa+get-4697@lists.php.net to get a copy of this message | ||
On Sat, 15 Dec 2007 18:19:51 +0100, cweiske@cweiske.de (Christian
Weiske) wrote:
>Hello Jorrit,
>
>
>> - In CVS there is a file called FTP_PHP5.php, which was an early
>> attempt of a maintainer before me to create a PHP5 version. It will
>> not be updated anymore and it causes the CS stats to show a high
>> number of errors, which is not nice. Shall I just remove it from CVS?
>If you really don't plan to use it anymore, remove it.
>
>IMO a php5 only version would be Net_FTP2 anyway.
Okay, that has been done. Now I'm down to only a couple of warnings,
but as discussed before, they can't be removed:
http://pear.cweiske.de/codingstandards/Net_FTP.html
I see that the script already filters the warnings about
PHPUnit_MAIN_METHOD not being in uppercase, so perhaps this error
could be filtered as well? I hate not being able to get the number of
errors down to 0 :)
Have a nice Sunday everybody,
Jorrit