- Issue created by @donquixote
If there are promoted properties, then I think it should be a convention to have each parameter on its own line. It's really helpful for readability. Since autowiring and other PHP attributes may be added as well, it would be far too long and unreadable to have parameters all in the same line.
- 🇳🇱Netherlands bbrala Netherlands
This issue was discussed in #3458726: Coding Standards Meeting Tuesday 2024-07-16 2100 UTC → .
We seem to be positive on this, but the IS needs some love.
Also mentioned was this rule: https://github.com/slevomat/coding-standard/issues/1684
- 🇦🇹Austria drunken monkey Vienna, Austria
+1
Good idea. I’d also support making this required for new code, at least for the first two “strong recommendations”. By supplying a maximum line length (e.g., 120 characters) we could even make the third one a hard rule, too. (I couldn’t really be enforced with just “very long” as the criterion.)
- 🇳🇿New Zealand quietone
This needs work on the proposed text section. At least, the comments there need to move elsewhere.
- Status changed to Needs work
11 months ago 5:14am 23 July 2024 - 🇩🇪Germany donquixote
At least, the comments there need to move elsewhere.
Not sure what you mean by that.
To me it seems clear that the proposed text is inside the quote, and what is below is comments.I am adding a separator heading "Notes", I hope this helps.
- 🇩🇪Germany donquixote
I’d also support making this required for new code, at least for the first two “strong recommendations”.
I would be ok with that.
I was hoping that something weaker would be more likely to land, but I have no objection to hard rules :)By supplying a maximum line length (e.g., 120 characters) we could even make the third one a hard rule, too. (I couldn’t really be enforced with just “very long” as the criterion.)
Ok for me, but this could broaden the scope quite a lot.