global: Remove leading/trailing blank lines in files #20758
Conversation
ulfalizer
requested review from
MaureenHelm,
aescolar,
albertofloyd,
anangl,
andrewboie,
andyross,
avisconti,
carlescufi,
ceolin,
cvinayak,
dbkinder,
dcpleung,
erwango,
finikorg,
franciscomunoz,
galak,
henrikbrixandersen,
ioannisg,
jakub-uC,
jfischer-no,
jhedberg,
joerchan,
jukkar,
nashif,
nordic-krch,
nvlsianpu,
pfalcon,
rlubos,
ruuddw and
scottwcpg
as code owners
|
Marking this TSC to see if we want to merge this for the release. |
|
As per TSC meeting there is no one actively requesting this PR to be merged during the feature freeze window, so it will be deferred until 2.1 is released. |
|
Is it a (new) requirement not to have leading/trailing blank lines in these files? We should document that in our coding/documenting guidelines (and enforce it as we do for trailing white-space on lines). |
Think it's usually a typo, so don't know how much it would help to document it. zephyrproject-rtos/ci-tools#112 will flag it automatically. |
henrikbrixandersen
removed request for
henrikbrixandersen,
jakub-uC,
rlubos,
scottwcpg,
thoh-ot and
wentongwu
Remove leading/trailing blank lines in .c, .h, .py, .rst, .yml, and .yaml files. Will avoid failures with the new CI test in zephyrproject-rtos/ci-tools#112, though it only checks changed files. Move the 'target-notes' target in boards/xtensa/odroid_go/doc/index.rst to get rid of the trailing blank line there. It was probably misplaced. Signed-off-by: Ulf Magnusson <[email protected]>
ulfalizer
force-pushed
the
leading-trailing-blanks
branch
from
f839df9 to
e5b5bd8
Compare
|
Rebased now. |
21 participants
Remove leading/trailing blank lines in .c, .h, .py, .rst, .yml, and
.yaml files.
Will avoid failures with the new CI test in
zephyrproject-rtos/ci-tools#112, though it only
checks changed files.
Move the 'target-notes' target in boards/xtensa/odroid_go/doc/index.rst
to get rid of the trailing blank line there. It was probably misplaced.