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

Extra whitespace for classes declaration #57

Closed
Zifius opened this issue May 25, 2012 · 2 comments
Closed

Extra whitespace for classes declaration #57

Zifius opened this issue May 25, 2012 · 2 comments

Comments

@Zifius
Copy link

Zifius commented May 25, 2012

I'm not much familiar with PSR(s), so I'd like to ask - when class declaration is padded with 4 spaces (or more), they are not removed. Example:

    class Test
    {
    }

should it be fixed to become:

class Test
{
}

Thanks!

@Seldaek
Copy link
Contributor

Seldaek commented Jun 25, 2012

I don't think this makes sense, there are a few legit use cases for having it indented (if it's in a if() block for some reason), and in general I don't think it's a kind of "small mistake" that can just be accidental.

@keradus
Copy link
Member

keradus commented Sep 24, 2014

I agree, we can't guess to change that indent or not.
CLosing after 2 years of silence.

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

No branches or pull requests

3 participants