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

Feature request: Make spacing between class properties configurable #60

Closed
Jeroeny opened this issue May 27, 2020 · 1 comment
Closed

Comments

@Jeroeny
Copy link

@Jeroeny Jeroeny commented May 27, 2020

I've noticed that v3.4.0 contains the following change: Printer: one-line properties without comments are not separated using one empty line.

In our codebase we have a code standard where we do have an empty line between properties, regardless of there being a comment or not. Now of course I understand that everyone has their own standards.

So maybe it could be nice if this value is configurable, just like linesBetweenMethods. So we can override it in our Printer.

I hope this is something that can be considered. If it is, I'd be happy to contribute a PR.

@dg dg closed this as completed in 995ed5b May 27, 2020
@dg
Copy link
Member

@dg dg commented May 27, 2020

Fixed, I added $lineBetweenProperties

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

2 participants