Julien Falque

Results 149 comments of Julien Falque

While I like the idea, given the current lack of activity on the repository, won't that close a lot of issues we actually want to fix?

Maybe we can start with higher delays to see whether it closes too many legit issues? What about 3 years?

I could not reproduce the bug. Please try again with latest `master` and, if the bug still happens, reopen this issue with all asked details to help us reproduce it.

I'm not sure I see value in ignoring non-monolithic files altogether. First, I don't think using PHP as a template language is inherently bad. Secondly, as @SpacePossum said, some rules...

Keeping commented-out code is a bad practice IMO. I don't want it to be supported by the tool.

> I don't say to activate it by default. The default value doesn't matter. If we want to have such option, we need to implement it and maintain it in...

This part should be fixed by #4884.

I think this is fixed via the new `statement_indentation` rule. Can you try installing `dev-master` and enabling this rule?

This seems to be caused by `blank_line_before_statement`. Can you try changing the config of the rule (to not add blank lines before `require_once` statements) or disabling it?