Skip to content

tree-wide: replace multiple empty lines with one#21502

Merged
maribu merged 1 commit intoRIOT-OS:masterfrom
maribu:tree-wide/replace-multiple-empty-lines-with-one
May 21, 2025
Merged

tree-wide: replace multiple empty lines with one#21502
maribu merged 1 commit intoRIOT-OS:masterfrom
maribu:tree-wide/replace-multiple-empty-lines-with-one

Conversation

@maribu
Copy link
Copy Markdown
Member

@maribu maribu commented May 21, 2025

Contribution description

For each C source/header $file: sed -e '/^$/N;/^\n$/D' -i $file.

Testing procedure

Only white-space changes

Issues/PRs references

None

@maribu
tree-wide: replace multiple empty lines with one
cac44ed 
For each C source/header `$file`: `sed -e '/^$/N;/^\n$/D' -i $file`.
@github-actions github-actions bot added Platform: native Platform: This PR/issue effects the native platform Platform: MSP Platform: This PR/issue effects MSP-based platforms Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: network Area: Networking Area: tests Area: tests and testing framework Platform: AVR Platform: This PR/issue effects AVR-based platforms Area: drivers Area: Device drivers Area: BLE Area: Bluetooth Low Energy support Area: timers Area: timer subsystems Area: arduino API Area: Arduino wrapper API labels May 21, 2025
@maribu maribu added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed Platform: AVR Platform: This PR/issue effects AVR-based platforms Area: drivers Area: Device drivers Area: BLE Area: Bluetooth Low Energy support Area: timers Area: timer subsystems Area: arduino API Area: Arduino wrapper API Area: LoRa Area: LoRa radio support Area: boards Area: Board ports Area: SAUL Area: Sensor/Actuator Uber Layer Area: OTA Area: Over-the-air updates Platform: ESP Platform: This PR/issue effects ESP-based platforms Platform: RISC-V Platform: This PR/issue effects RISC-V-based platforms Area: CoAP Area: Constrained Application Protocol implementations Area: cpu Area: CPU/MCU ports Area: USB Area: Universal Serial Bus Area: sys Area: System Area: examples Area: Example Applications labels May 21, 2025
@maribu maribu mentioned this pull request May 21, 2025
Merged
@riot-ci
Copy link
Copy Markdown

riot-ci commented May 21, 2025

Murdock results

✔️ PASSED

cac44ed tree-wide: replace multiple empty lines with one

Success Failures Total Runtime
1 0 1 02m:16s

Artifacts

@maribu
Copy link
Copy Markdown
Member Author

maribu commented May 21, 2025

Note that Murdock did not run any builds, because the can_fast_ci_run.py script detected that the changes where white-space only. This implicitly confirms that this PR does only touch white space.

@maribu maribu added this pull request to the merge queue May 21, 2025
Merged via the queue into RIOT-OS:master with commit 09c30e8 May 21, 2025
50 checks passed
@crasbe crasbe mentioned this pull request May 21, 2025
Closed
8 tasks
@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented May 21, 2025

Wow, that was one quick shrinking of the codebase :D

@maribu maribu deleted the tree-wide/replace-multiple-empty-lines-with-one branch May 21, 2025 22:08
@gschorcht gschorcht mentioned this pull request May 21, 2025
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gschorcht gschorcht gschorcht approved these changes

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

@bergzand bergzand Awaiting requested review from bergzand bergzand is a code owner

@PeterKietzmann PeterKietzmann Awaiting requested review from PeterKietzmann

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@basilfx basilfx Awaiting requested review from basilfx basilfx is a code owner

@jue89 jue89 Awaiting requested review from jue89 jue89 is a code owner

@jeandudey jeandudey Awaiting requested review from jeandudey jeandudey is a code owner

@benpicco benpicco Awaiting requested review from benpicco benpicco is a code owner

@vincent-d vincent-d Awaiting requested review from vincent-d vincent-d is a code owner

@dylad dylad Awaiting requested review from dylad dylad is a code owner

@keestux keestux Awaiting requested review from keestux keestux is a code owner

@nandojve nandojve Awaiting requested review from nandojve nandojve is a code owner

@kYc0o kYc0o Awaiting requested review from kYc0o kYc0o is a code owner

Assignees

No one assigned

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Milestone

Release 2025.07

Development

Successfully merging this pull request may close these issues.

5 participants

@maribu @riot-ci @crasbe @gschorcht @Teufelchen1