Skip to content

Conversation

@wep21
Copy link

@wep21 wep21 commented Aug 7, 2022

Signed-off-by: Daisuke Nishimatsu [email protected]

Description

update rclcpp action api for humble/rolling
This api change was introduced by ros2/rclcpp#1311

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce. Also list any relevant details for your test configuration.

# Run this command
ros2 launch package launch.py

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

@wep21 wep21 requested a review from roni-kreinin as a code owner August 7, 2022 06:11
@wep21 wep21 force-pushed the update-action-api branch from 928b38e to e8cfed7 Compare August 7, 2022 06:11
@wep21
Copy link
Author

wep21 commented Aug 7, 2022

@roni-kreinin Could you create humble or rolling branch? I would like to change the base branch after the branch is created.

@roni-kreinin
Copy link
Contributor

humble branch created. Will this change need to be applied to galactic too?

@wep21
Copy link
Author

wep21 commented Aug 8, 2022

The old api is compatible, but is marked as deprecated even in galactic. https://github.com/ros2/rclcpp/blob/206e0fd4fefb9fcf69d42c03ffbefd347f110aae/rclcpp_action/include/rclcpp_action/client.hpp#L289-L299
It may be better to add the changes into galactic too.

@wep21 wep21 force-pushed the update-action-api branch from e8cfed7 to 8a9d407 Compare August 9, 2022 02:20
Signed-off-by: Daisuke Nishimatsu <[email protected]>
@wep21 wep21 force-pushed the update-action-api branch from 8a9d407 to bafa128 Compare August 9, 2022 02:35
@roni-kreinin
Copy link
Contributor

Merging for galactic and will keep this API for humble.

@roni-kreinin roni-kreinin merged commit cc4a5e2 into turtlebot:galactic Aug 9, 2022
@wep21 wep21 deleted the update-action-api branch August 10, 2022 03:52
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