Skip to content

Add battery_state_broadcaster (backport #2086) - #2521

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

Add battery_state_broadcaster (backport #2086)#2521
christophfroehlich merged 1 commit into
jazzyfrom
mergify/bp/jazzy/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)
@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.01%. Comparing base (2d3f0f4) to head (6c5d524).

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             @@
##            jazzy    #2521      +/-   ##
==========================================
- Coverage   86.04%   86.01%   -0.03%     
==========================================
  Files         159      163       +4     
  Lines       16043    16428     +385     
  Branches     1401     1444      +43     
==========================================
+ Hits        13804    14131     +327     
- Misses       1732     1763      +31     
- Partials      507      534      +27     
Flag Coverage Δ
unittests 86.01% <84.67%> (-0.03%) ⬇️

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%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich
christophfroehlich merged commit 3dbebf0 into jazzy Aug 4, 2026
18 checks passed
@christophfroehlich
christophfroehlich deleted the mergify/bp/jazzy/pr-2086 branch August 4, 2026 04:17
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