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

Use one blank line after the opening tag and namespace declaration #20379

Merged
merged 2 commits into from
Apr 9, 2020

Conversation

ChristophWurst
Copy link
Member

Another step to bring us closer to PSR2.

For PSR2

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@nickvergessen nickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 9, 2020
@georgehrke
Copy link
Member

Do we also have a standard for number of lines between copyright block and namespace?

@ChristophWurst
Copy link
Member Author

Do we also have a standard for number of lines between copyright block and namespace?

Not yet, I think. But IIRC there is also an PSR for the order of declare, header, namespace and so on.

@ChristophWurst ChristophWurst merged commit 19e97e8 into master Apr 9, 2020
@ChristophWurst ChristophWurst deleted the techdebt/blank-lines branch April 9, 2020 11:44
@rullzer rullzer mentioned this pull request Apr 9, 2020
59 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants