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 output mode configured in ValueGenerator instance #174

Merged
merged 2 commits into from
Mar 8, 2023

Conversation

thomas-kl1
Copy link
Contributor

Q A
Documentation no
Bugfix yes
BC Break no
New Feature no
RFC no
QA no

Description

When I want to generate code for an array with the "single line" output, I expect the result to be on a single line. However in the case the array have deeper depth than 1, the result is on multi lines, because the original output mode is not given when generating the child nodes.

Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a test

@thomas-kl1 thomas-kl1 force-pushed the patch-1 branch 3 times, most recently from 10ad5d9 to 40df8f5 Compare March 7, 2023 22:39
Signed-off-by: Thomas Klein <thomasklein876@gmail.com>
@thomas-kl1
Copy link
Contributor Author

Hi @Ocramius
I've pushed the test as well. Do you want me to add a suppress warning for the psalm rule "unused method" on the data provider?

Signed-off-by: Thomas Klein <thomasklein876@gmail.com>
@Ocramius Ocramius self-assigned this Mar 8, 2023
@Ocramius Ocramius changed the title Use output mode configured in ValueGenerator instance Use output mode configured in ValueGenerator instance Mar 8, 2023
@Ocramius Ocramius added this to the 4.10.0 milestone Mar 8, 2023
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @thomas-kl1!

@Ocramius Ocramius merged commit ad8b360 into laminas:4.10.x Mar 8, 2023
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.

3 participants