Julien Falque

Results 149 comments of 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?

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?