Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.0.0a1

Request #19843 Add parameter to Squiz.WhiteSpace.FunctionSpacing sniff
Submitted: 2013-03-06 17:58 UTC Modified: 2013-03-26 00:21 UTC
From: aik099 Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.4.4)
PHP Version: 5.4.6 OS: Linux
Roadmaps: (Not assigned)    
Subscription  


 [2013-03-06 17:58 UTC] aik099 (Alexander Obuhovich)
Description: ------------ Right now "Squiz.WhiteSpace.FunctionSpacing" sniff expects 2 blank lines between 2 functions. This might be good on it's own, but in our project we need 1 line spacing. I'm proposing to add "functionSpacing" parameter to a sniff (2 by default for backwards compatibility).

Comments

 [2013-03-26 00:21 UTC] squiz (Greg Sherwood)
-Status: Open +Status: Closed -Assigned To: +Assigned To: squiz
I've committed a new setting to the github repo: https://github.com/squizlabs/PHP_CodeSniffer/commit/8e1e523592a180c63fe4c8f4 704f6ca5d787d525 You can include the sniff in your ruleset, and override the spacing setting, like this: <rule ref="Squiz.WhiteSpace.FunctionSpacing"> <properties> <property name="spacing" value="1"/> </properties> </rule> Let me know if you have any problems.