Skip to content

Comments

Fix goal handle was not freed correctly (backport #4137)#4142

Merged
SteveMacenski merged 1 commit intoironfrom
mergify/bp/iron/pr-4137
Feb 23, 2024
Merged

Fix goal handle was not freed correctly (backport #4137)#4142
SteveMacenski merged 1 commit intoironfrom
mergify/bp/iron/pr-4137

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Feb 23, 2024

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


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* Fix goal handle was not freed correctly

* Update nav2_util/include/nav2_util/simple_action_server.hpp

Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

---------

Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
Co-authored-by: Ziohang <hang.zhou@agvsz.com>
Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>
(cherry picked from commit 2e491b1)
@mergify
Copy link
Contributor Author

mergify bot commented Feb 23, 2024

@mergify[bot], all pull requests must be targeted towards the main development branch.
Once merged into main, it is possible to backport to @iron, but it must be in main
to have these changes reflected into new distributions.

@SteveMacenski SteveMacenski merged commit 6003c10 into iron Feb 23, 2024
@SteveMacenski SteveMacenski deleted the mergify/bp/iron/pr-4137 branch February 23, 2024 16:58
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