Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

blank lines after methods but not last one #16

Open
ofbeaton opened this issue Jul 27, 2015 · 1 comment
Open

blank lines after methods but not last one #16

ofbeaton opened this issue Jul 27, 2015 · 1 comment

Comments

@ofbeaton
Copy link
Owner

It would be nice if Squiz.WhiteSpace.FunctionSpacing.After would not require a blank line after a method if it is the last method in a class.

class A 
{
  public function method1()
  {
  }

  public function method2()
  {
  }

  public function method2()
  {
  }
}

This would be much nicer than enforcing a space after the last method. You could switch it to enforce a space before a method, but then you'd run into the same problem but at the top (assuming no properties).

Request this upstream but if they don't want to do it, implement it ourselves.

@ofbeaton
Copy link
Owner Author

squizlabs/PHP_CodeSniffer#652

requested upstream.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant