search

php-stubs / wordpress-stubs

Up-to-date WordPress function and class declaration stubs for static analysis by PHPStan
https://packagist.org/packages/php-stubs/wordpress-stubs
MIT License
146 stars 18 forks source link

Temporarily configure Squiz.WhiteSpace.FunctionSpacing #235

Closed IanDelMar closed 5 days ago

IanDelMar commented 6 days ago

See https://github.com/szepeviktor/phpcs-psr-12-neutron-hybrid-ruleset/pull/31

szepeviktor commented 5 days ago

Do you have an idea how to shrink that 63-line method?

https://github.com/szepeviktor/phpcs-psr-12-neutron-hybrid-ruleset/blob/a3392953a337ef3bfa09d546a070ac2db4dfcddd/PSR12NeutronRuleset/ruleset.xml#L256

IanDelMar commented 5 days ago

Right now, no, except for moving parts of the methods into another method. I suggest opening an issue if you are concerned about the 63-line method; but, we have been ignoring such long methods in the past. #238