Skip to content

Conversation

@ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Jan 26, 2024

Related with this PR ros2/rclcpp#2413

Added note about rclcpp/qos_event.hpp removal in Jazzy

Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

The note looks great, but if you make the change to #4138 (review) , this will have a conflict and need a rebase.

@ahcorde ahcorde requested a review from clalancette January 26, 2024 17:50
@ahcorde
Copy link
Contributor Author

ahcorde commented Jan 26, 2024

@clalancette rebase done.

@clalancette clalancette added the backport-all backport at reviewers discretion; from rolling to all versions label Jan 26, 2024
@clalancette clalancette merged commit c3e44bf into rolling Jan 26, 2024
@delete-merged-branch delete-merged-branch bot deleted the ahcorde/jazzy/qos_event_deprecation_remove branch January 26, 2024 17:53
mergify bot pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit c3e44bf)
mergify bot pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit c3e44bf)
clalancette pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit c3e44bf)

Co-authored-by: Alejandro Hernández Cordero <[email protected]>
clalancette pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit c3e44bf)

Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-all backport at reviewers discretion; from rolling to all versions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants