As there is no coding standard saying there should be a blank line between <?php and /**, I'd like to propose that all the empty lines following <?php should be removed (and probably blank lines at the beginning of text files as well).

CommentFileSizeAuthor
#1 1078888-01-remove_cruft_blank_lines.patch167.53 KBFreso

Comments

Freso’s picture

Freso commented
Status: Active » Needs review
StatusFileSize
new 1078888-01-remove_cruft_blank_lines.patch167.53 KB

And after some sed magic (basically sed -i '1,2{/^$/d}') with manual QA, here's a patch.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Version: 7.x-dev » 8.x-dev

I think that having 1 empty line is good for readability

aspilicious’s picture

aspilicious commented

Mèh, I always add a blank line for the reason posted in #2.

-1 for me

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Needs review » Closed (won't fix)

Also think that empty line puts a better emphasis on the @file docs. Reopen if you disagree.