Julien Falque
Julien Falque
This creates a [conflict](https://github.com/FriendsOfPHP/PHP-CS-Fixer/runs/7369775808?check_suite_focus=true#step%3A11%3A517=) with [PSR-12 rule about the `// no break` comment](https://www.php-fig.org/psr/psr-12/#52-switch-case). Should the comment remain indented as part of the case body? Or should it be indented like...
Thinking about this again. Currently this is supported for `else`/`elseif` blocks but only to make tests from `braces` pass. I think there is no reliable way to say whether a...
@SpacePossum @keradus @kubawerlos What is your opinion about this?
We already have `allow_single_line_anonymous_class_with_empty_body` which is similar and maybe there are more situations where that could make sense, e.g. empty `catch` block. Shall we aim for a more generic option?
This should be fixed by #4884.
Reported issue was fixed by #4884. @Seldaek this might be fixed by #6490 depending on its outcome.
I think that `null` being implicitly allowed because of the default value can be misleasing, I always prefer when things are explicit.
The rule works as expected when I try it with your snippet. Are you certain the file does not already end with a newline?
I didn't see integration tests in #6409 regarding this issue. @SpacePossum are we sure this is fixed?