Re: PHP_CodeSniffer 1.2
| From: | Ken Guest | Date: | Thu, 05 Nov 2009 17:40:36 +0000 |
| Subject: | Re: PHP_CodeSniffer 1.2 | ||
| References: | 1 2 3 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-53032@lists.php.net to get a copy of this message | ||
this makes me think of the php_beautifier package.
if there were a way of getting it to accept errors & warnings from
phpcs and modify the tested code so that it does adhere to the
standards that'd be amazing.
k.
On Thu, Nov 5, 2009 at 5:29 PM, Bill Shupp <hostmaster@shupp.org> wrote:
> On Nov 5, 2009, at 12:53 AM, Christian Weiske wrote:
>
>> Hi Bill,
>>
>>
>> The correct way is:
>> public function directRequest($url,
>> OpenID_Message $message, array $options = array()
>> ) {
>> ....
>> }
>>
>> see: http://pear.php.net/manual/en/standards.funcdef.php
>> "Split function definitions onto several lines"
>
> I've never noticed this, and don't like it. I think it's much harder to
> read this way than splitting them on one argument per line. And now, after
> I've spent lots of time getting my code to pass phpcs 1.1 cleanly in my
> OpenID package, and now with the upgrade to 1.2, I'm left with 475 warnings
> or errors to deal with.
>
> Bill
>
> --
> PEAR Development Mailing List (http://pear.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
http://blogs.linux.ie/kenguest/