Re: PHP_CodeSniffer 1.2

From: 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/

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