ros2 / demos

Apache License 2.0
502 stars 330 forks source link

Update usage of `spin_until_future_complete` #556

Closed hliberacki closed 6 months ago

hliberacki commented 2 years ago

Use spin_until_complete instead of deprecated spin_until_future_complete

Signed-off-by: Hubert Liberacki hliberacki@gmail.com

hliberacki commented 2 years ago

Due to changes in RCLCPP, made in https://github.com/ros2/rclcpp/pull/1874 PR.

audrow commented 2 years ago
hliberacki commented 2 years ago

@audrow https://github.com/ros2/rclcpp/pull/1874#issuecomment-1086906375 Passing CI with all related PRs linked and build together.

christophebedard commented 7 months ago

I've opened #669 to replace this PR.

christophebedard commented 6 months ago

This can be closed, see https://github.com/ros2/demos/pull/669#issuecomment-2035715274