Skip to content

Conversation

@SteveMacenski
Copy link
Member

No description provided.

Signed-off-by: Steve Macenski <[email protected]>
Signed-off-by: Steve Macenski <[email protected]>
@SteveMacenski SteveMacenski marked this pull request as ready for review July 10, 2025 15:39
@SteveMacenski
Copy link
Member Author

@claude open branches to do this in Jazzy and Humble

@claude
Copy link
Contributor

claude bot commented Jul 10, 2025

Claude finished @SteveMacenski's task —— View job


Creating Branches for Jazzy and Humble Backports

Todo List:

  • Analyze current changes in the PR
  • Create branch for Jazzy backport
  • Apply changes to Jazzy branch
  • Create branch for Humble backport
  • Apply changes to Humble branch
  • Verify all changes are committed and pushed

Changes to backport:

  • Change std::unique_ptr<bond::Bond> to std::shared_ptr<bond::Bond> in header file
  • Change std::make_unique<bond::Bond> to std::make_shared<bond::Bond> in implementation

Results:

Jazzy branch created and committed (SHA: c761cf9)
Humble branch created and committed (SHA: 5bcfa6f)

Both branches have been successfully created with the bond connection changes backported. The changes modify the bond connection from using std::unique_ptr to std::shared_ptr for better memory management and lifecycle handling in the Navigation2 lifecycle nodes.


claude bot and others added 2 commits July 10, 2025 15:59
Changes bond connection from std::unique_ptr to std::shared_ptr
for better memory management and lifecycle handling.

Co-authored-by: Steve Macenski <[email protected]>
Changes bond connection from std::unique_ptr to std::shared_ptr
for better memory management and lifecycle handling.

Co-authored-by: Steve Macenski <[email protected]>
@SteveMacenski SteveMacenski merged commit 77db0a6 into kilted Jul 10, 2025
8 of 10 checks passed
@SteveMacenski SteveMacenski deleted the SteveMacenski-patch-1 branch July 10, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants