Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Apr 29, 2025

closes #2795

Note

extend_name_with_sub_namespace is inline function, so user must rebuild the application with rclcpp to get this fix. besides that, this is behavior change could break the application. i am not inclined to backport this. thoughts?


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

* subordinate node consistent behavior and update docstring.

Signed-off-by: Tomoya Fujita <[email protected]>

* add subnode_parameter_operation test.

Signed-off-by: Tomoya Fujita <[email protected]>

* typo fixes.

Signed-off-by: Tomoya Fujita <[email protected]>

---------

Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit c89a2b1)
@fujitatomoya
Copy link
Collaborator

Pulls: #2830
Gist: https://gist.githubusercontent.com/fujitatomoya/206bd7249eb803af5ca89876e6adc3a3/raw/3a3dab11c3117ee02c9acc2ecf7d6ef6348f6e6b/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: kilted
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/15838

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

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