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
SingleSpaceAfterConstructFixer - better handling of closing parenthesis and brace #5453
Conversation
'<?php | ||
new Dummy(/* a */); | ||
new Dummy(/** b */); | ||
foo(/* c */); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about foo('x'/* c */, 'y');
?
I think it still enforces space after */
which is againt every convention
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also for block comment before ->
, see https://github.com/atk4/data/pull/827/files#diff-1b13a61af17b481a4e15159a87f2a89937836e504284460f2164bfafc1d32348R83
closes #5448