Skip to content

Added note about rclcpp/qos_event.hpp removal in Jazzy#4137

Merged
clalancette merged 2 commits intorollingfrom
ahcorde/jazzy/qos_event_deprecation_remove
Jan 26, 2024
Merged

Added note about rclcpp/qos_event.hpp removal in Jazzy#4137
clalancette merged 2 commits intorollingfrom
ahcorde/jazzy/qos_event_deprecation_remove

Conversation

@ahcorde
Copy link
Copy Markdown
Collaborator

@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 <ahcorde@gmail.com>
Copy link
Copy Markdown
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
Copy Markdown
Collaborator 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 <ahcorde@gmail.com>
(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 <ahcorde@gmail.com>
(cherry picked from commit c3e44bf)
clalancette pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit c3e44bf)

Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
clalancette pushed a commit that referenced this pull request Jan 26, 2024
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit c3e44bf)

Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
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.

2 participants