Skip to content

Add battery_state_broadcaster (backport #2086) - #2522

Merged
christophfroehlich merged 1 commit into
kiltedfrom
mergify/bp/kilted/pr-2086
Aug 4, 2026
Merged

Add battery_state_broadcaster (backport #2086)#2522
christophfroehlich merged 1 commit into
kiltedfrom
mergify/bp/kilted/pr-2086

Conversation

@mergify

@mergify mergify Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This moves https://github.com/ipa320/ros_battery_monitoring/tree/main/battery_state_broadcaster to our repository.

For discsussion about this, see #1888 and especially #1888 (comment)


This is an automatic backport of pull request #2086 done by Mergify.

Co-authored-by: Jonas Otto <jonas.otto@ipa.fraunhofer.de>
Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Co-authored-by: Jonas Otto <jonas@jonasotto.com>
Co-authored-by: Michal Sojka <michal.sojka@cvut.cz>
Co-authored-by: Yara Shahin <58101871+YaraShahin@users.noreply.github.com>
Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
Co-authored-by: Sai Kishor Kothakota <sai.kishor@pal-robotics.com>
(cherry picked from commit 5b2e1ca)
@mergify

mergify Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@christophfroehlich christophfroehlich added the check-prerelease Runs the pre-release workflow without downstream dependencies label Aug 3, 2026
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.67532% with 59 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.88%. Comparing base (0c16808) to head (10dc0e0).

Files with missing lines Patch % Lines
...tate_broadcaster/src/battery_state_broadcaster.cpp 77.54% 29 Missing and 24 partials ⚠️
...roadcaster/test/test_battery_state_broadcaster.hpp 92.40% 1 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           kilted    #2522      +/-   ##
==========================================
- Coverage   86.93%   86.88%   -0.06%     
==========================================
  Files         157      161       +4     
  Lines       15746    16131     +385     
  Branches     1342     1385      +43     
==========================================
+ Hits        13689    14015     +326     
- Misses       1586     1616      +30     
- Partials      471      500      +29     
Flag Coverage Δ
unittests 86.88% <84.67%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...roadcaster/test/test_battery_state_broadcaster.cpp 100.00% <100.00%> (ø)
...aster/test/test_load_battery_state_broadcaster.cpp 100.00% <100.00%> (ø)
...roadcaster/test/test_battery_state_broadcaster.hpp 92.40% <92.40%> (ø)
...tate_broadcaster/src/battery_state_broadcaster.cpp 77.54% <77.54%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich
christophfroehlich merged commit bc333b7 into kilted Aug 4, 2026
17 of 18 checks passed
@christophfroehlich
christophfroehlich deleted the mergify/bp/kilted/pr-2086 branch August 4, 2026 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

check-prerelease Runs the pre-release workflow without downstream dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant