Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

suggest spin_all to migrate spinOnce to execute all available works i… #4966

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

fujitatomoya
Copy link
Collaborator

…n the queue.

closes #4959

Copy link

HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/12979266704/artifacts/2487880655.

To view the resulting site:

  1. Click on the above link to download the artifacts archive
  2. Extract it
  3. Open html-artifacts-4966/index.html in your favorite browser

@ahcorde ahcorde added the backport-jazzy backport at reviewers discretion; from rolling to jazzy label Jan 27, 2025
@ahcorde ahcorde merged commit 482b754 into rolling Jan 27, 2025
4 checks passed
@ahcorde ahcorde deleted the fujitatomoya/update-migration-cpp-example branch January 27, 2025 08:45
mergify bot pushed a commit that referenced this pull request Jan 27, 2025
…n the queue. (#4966)

Signed-off-by: Tomoya.Fujita <[email protected]>
(cherry picked from commit 482b754)
ahcorde pushed a commit that referenced this pull request Jan 27, 2025
…n the queue. (#4966) (#4969)

Signed-off-by: Tomoya.Fujita <[email protected]>
(cherry picked from commit 482b754)

Co-authored-by: Tomoya Fujita <[email protected]>
@fujitatomoya
Copy link
Collaborator Author

@Mergifyio backport humble.

note that this needs to be backported for humble, ros2/rclcpp#2508 was regression introduced in jazzy development.

Copy link
Contributor

mergify bot commented Jan 27, 2025

backport humble.

❌ Sorry but I didn't understand the arguments of the command backport. Please consult the commands documentation 📚.

@fujitatomoya
Copy link
Collaborator Author

@Mergifyio backport humble.

fujitatomoya added a commit that referenced this pull request Jan 27, 2025
fujitatomoya added a commit that referenced this pull request Jan 27, 2025
@kscottz
Copy link
Collaborator

kscottz commented Jan 27, 2025

Late to the game, but thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-jazzy backport at reviewers discretion; from rolling to jazzy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ROS2 migration guide replaces spinOnce with spin_some
4 participants