Skip to content

Fix wrapping array because the property name length [Closes #45] - #47

Merged
dg merged 4 commits into
nette:masterfrom
zeleznypa:master
Nov 19, 2019
Merged

dg merged 4 commits into
nette:masterfrom
zeleznypa:master

Conversation

@zeleznypa

Copy link
Copy Markdown
Contributor

Comment thread src/PhpGenerator/Helpers.php
Comment thread src/PhpGenerator/Printer.php
Comment thread tests/PhpGenerator/Helpers.dump().phpt
Comment thread tests/PhpGenerator/Helpers.dump().phpt Outdated
@adaamz

adaamz commented Nov 15, 2019

Copy link
Copy Markdown
Contributor

hm, i don't see any \n in the cs failing code

@zeleznypa

Copy link
Copy Markdown
Contributor Author

hm, i don't see any \n in the cs failing code

It's fixed. :) Thanks

@dg
dg force-pushed the master branch 3 times, most recently from 29003f5 to df381c7 Compare November 19, 2019 12:11
@dg

dg commented Nov 19, 2019

Copy link
Copy Markdown
Member

Thanks!

@dg
dg merged commit 1b954ea into nette:master Nov 19, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
dg pushed a commit that referenced this pull request Nov 20, 2019
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

Successfully merging this pull request may close these issues.

Wrong wrapping condition in array formater

3 participants