Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 21, 2025
Merged

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

merged 1 commit into from
May 21, 2025

Conversation

maribu
Copy link
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 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

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
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
Open
8 tasks
@crasbe
Copy link
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
@Teufelchen1 Teufelchen1 added this to the Release 2025.07 milestone Jul 14, 2025
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 PeterKietzmann is a code owner

@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