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

Document gracefull cancellation API #535

Conversation

Rayman
Copy link
Contributor

@Rayman Rayman commented Mar 25, 2024

Merge after ros-navigation/navigation2#4136 is merged

Copy link
Member

@SteveMacenski SteveMacenski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add to migration guide - this is a good new features for folks to be aware of :-)

https://navigation.ros.org/migration/Iron.html

@Rayman Rayman force-pushed the feature/4033/add-graceful-cancellation-api branch from e961ff5 to 2704a03 Compare March 26, 2024 12:17
@Rayman
Copy link
Contributor Author

Rayman commented Mar 26, 2024

I've added it to the migration guide

migration/Iron.rst Outdated Show resolved Hide resolved
@SteveMacenski
Copy link
Member

@Rayman your fork doesn't let me commit to it with the suggestion, but after that can merge

@Rayman Rayman force-pushed the feature/4033/add-graceful-cancellation-api branch from 53a0ed4 to ab532af Compare March 28, 2024 09:52
@SteveMacenski SteveMacenski merged commit 4c1c9e6 into ros-navigation:master Mar 28, 2024
1 check passed
@Rayman Rayman deleted the feature/4033/add-graceful-cancellation-api branch March 29, 2024 10:02
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