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

Note about removed deprecated headers #4138

Merged
merged 3 commits into from
Jan 26, 2024

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Jan 26, 2024

Related with this PR ros2/geometry2#645

Note about removed deprecated headers

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

@ahcorde The note looks great.

I know you didn't do this, but would you mind moving this entire "New features in this ROS 2 release" section above, right under "Installation"? That will restore the flow that we usually have for these pages. Thanks in advance.

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from clalancette January 26, 2024 15:24
@ahcorde
Copy link
Contributor Author

ahcorde commented Jan 26, 2024

@clalancette sure, done here 468e4bf

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.

One minor change to the wording, then I think this is good to go. Thanks for doing the rearrangement!

Comment on lines 52 to 53
``tf2_kdl/tf2_kdl.h``, ``tf2_sensor_msgs/tf2_sensor_msgs.h`` headers been completely removed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
``tf2_kdl/tf2_kdl.h``, ``tf2_sensor_msgs/tf2_sensor_msgs.h`` headers been completely removed.
``tf2_kdl/tf2_kdl.h``, ``tf2_sensor_msgs/tf2_sensor_msgs.h`` headers have been completely removed.

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from clalancette January 26, 2024 17:02
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.

Looks good, thank you for iterating!

@clalancette clalancette added the backport-all backport at reviewers discretion; from rolling to all versions label Jan 26, 2024
@clalancette clalancette merged commit 7f5cc33 into rolling Jan 26, 2024
3 checks passed
@clalancette clalancette deleted the ahcorde/rolling/geometry_deprecated_remove branch January 26, 2024 17:48
mergify bot pushed a commit that referenced this pull request Jan 26, 2024
* Note about removed deprecated headers

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 7f5cc33)
mergify bot pushed a commit that referenced this pull request Jan 26, 2024
* Note about removed deprecated headers

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 7f5cc33)
clalancette pushed a commit that referenced this pull request Jan 26, 2024
* Note about removed deprecated headers

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 7f5cc33)

Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
clalancette pushed a commit that referenced this pull request Jan 26, 2024
* Note about removed deprecated headers

Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
(cherry picked from commit 7f5cc33)

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