Skip to content

Cleanup old internal API#2346

Merged
bmagyar merged 2 commits into
ros-controls:masterfrom
pal-robotics-forks:cleanup/old/internal/API
Jul 1, 2025
Merged

Cleanup old internal API#2346
bmagyar merged 2 commits into
ros-controls:masterfrom
pal-robotics-forks:cleanup/old/internal/API

Conversation

@saikishor

Copy link
Copy Markdown
Member

This PR cleans up old internal API that is unused after #2323 :)

@codecov

codecov Bot commented Jun 28, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.99%. Comparing base (2e82c40) to head (60b4f87).
Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2346   +/-   ##
=======================================
  Coverage   88.99%   88.99%           
=======================================
  Files         148      148           
  Lines       16805    16806    +1     
  Branches     1438     1438           
=======================================
+ Hits        14955    14956    +1     
  Misses       1287     1287           
  Partials      563      563           
Flag Coverage Δ
unittests 88.99% <100.00%> (+<0.01%) ⬆️

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

Files with missing lines Coverage Δ
controller_manager/src/controller_manager.cpp 74.48% <100.00%> (+0.01%) ⬆️
hardware_interface/src/resource_manager.cpp 78.26% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread hardware_interface/src/resource_manager.cpp

@bmagyar bmagyar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@bmagyar bmagyar merged commit cd7e522 into ros-controls:master Jul 1, 2025
18 of 26 checks passed
@bmagyar bmagyar deleted the cleanup/old/internal/API branch July 1, 2025 12:13
@saikishor saikishor added the backport-jazzy Triggers PR backport to ROS 2 jazzy. label Jul 1, 2025
mergify Bot pushed a commit that referenced this pull request Jul 1, 2025
(cherry picked from commit cd7e522)

# Conflicts:
#	controller_manager/src/controller_manager.cpp
#	hardware_interface/src/resource_manager.cpp
saikishor added a commit that referenced this pull request Jul 21, 2025
saikishor added a commit that referenced this pull request Jul 21, 2025
bmagyar pushed a commit that referenced this pull request Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-jazzy Triggers PR backport to ROS 2 jazzy.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants