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

fix(method_argument_space): inject new line after trailing space on current line #7327

Merged
merged 2 commits into from Sep 26, 2023

Conversation

SpacePossum
Copy link
Contributor

@SpacePossum SpacePossum commented Sep 26, 2023

closes #7291

The issues was perceived as a priority issue, however the issue is within the MethodArgumentSpaceFixer.
It creates a stream with multiple space-tokens which is not allowed.

@SpacePossum SpacePossum changed the title fix: MethodArgumentSpaceFixer inject new line after trailing space on… fix(method_argument_space): inject new line after trailing space on current line Sep 26, 2023
@coveralls
Copy link

coveralls commented Sep 26, 2023

Coverage Status

Changes unknown when pulling f7f4110 on SpacePossum:7291 into ** on PHP-CS-Fixer:master**.

@keradus keradus enabled auto-merge (squash) September 26, 2023 20:03
@keradus keradus merged commit 65ad67e into PHP-CS-Fixer:master Sep 26, 2023
15 checks passed
@SpacePossum SpacePossum deleted the 7291 branch September 26, 2023 20:33
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.

Two passes needed for method_argument_space + single_line_comment_style/no_trailing_whitespace
3 participants