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

Require space after Block Control Structures #137

Merged
merged 1 commit into from Nov 27, 2019
Merged

Require space after Block Control Structures #137

merged 1 commit into from Nov 27, 2019

Conversation

simPod
Copy link
Contributor

@simPod simPod commented Nov 26, 2019

based on top of #136

Addresses #135 (review)

@simPod simPod requested a review from a team as a code owner November 26, 2019 13:42
@simPod simPod mentioned this pull request Nov 26, 2019
tests/expected_report.txt Outdated Show resolved Hide resolved
@greg0ire

This comment has been minimized.

@simPod simPod changed the title Add control spacing sniff to require space after try, switch and if, foreach and for Require space after Block Control Structures Nov 26, 2019
@Ocramius Ocramius added this to the 7.0.0 milestone Nov 26, 2019
@greg0ire greg0ire merged commit 8372d9c into doctrine:master Nov 27, 2019
@greg0ire
Copy link
Member

Thanks @simPod !

@kukulich
Copy link
Contributor

@simPod btw the PR requires space even before block control structures.

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

Successfully merging this pull request may close these issues.

None yet

6 participants