Skip to content

Revert "Support loading multiple behavior tree files as subtrees" in Kilted#5907

Merged
SteveMacenski merged 1 commit intoros-navigation:kiltedfrom
mini-1235:kilted-patch-3
Jan 27, 2026
Merged

Revert "Support loading multiple behavior tree files as subtrees" in Kilted#5907
SteveMacenski merged 1 commit intoros-navigation:kiltedfrom
mini-1235:kilted-patch-3

Conversation

@mini-1235
Copy link
Collaborator

The PR was backported to Kilted in 47f84de, however, as reported in #5426, I think it is better to revert this commit in Kilted

…-navigation#5426)"

This reverts commit 57a639e.

Signed-off-by: mini-1235 <mauricepurnawan@gmail.com>
@mergify
Copy link
Contributor

mergify bot commented Jan 27, 2026

@mini-1235, all pull requests must be targeted towards the main development branch.
Once merged into main, it is possible to backport to @kilted, but it must be in main
to have these changes reflected into new distributions.

@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...clude/nav2_behavior_tree/bt_action_server_impl.hpp 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
...nclude/nav2_behavior_tree/behavior_tree_engine.hpp 100.00% <ø> (ø)
...ee/include/nav2_behavior_tree/bt_action_server.hpp 83.33% <ø> (ø)
nav2_behavior_tree/src/behavior_tree_engine.cpp 82.75% <ø> (-4.34%) ⬇️
...avigator/src/navigators/navigate_through_poses.cpp 88.23% <ø> (ø)
...2_bt_navigator/src/navigators/navigate_to_pose.cpp 83.33% <ø> (ø)
...core/include/nav2_core/behavior_tree_navigator.hpp 88.88% <ø> (-0.77%) ⬇️
...clude/nav2_behavior_tree/bt_action_server_impl.hpp 89.59% <0.00%> (+1.00%) ⬆️

... and 3 files with indirect coverage changes

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

@SteveMacenski SteveMacenski merged commit 66d3764 into ros-navigation:kilted Jan 27, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants