Skip to content

Remove deprecated rosidl_typesupport_cpp/type_support_map.h#167

Merged
christophebedard merged 1 commit intorollingfrom
christophebedard/remove-deprecated-type-support-map-h
Aug 15, 2025
Merged

Remove deprecated rosidl_typesupport_cpp/type_support_map.h#167
christophebedard merged 1 commit intorollingfrom
christophebedard/remove-deprecated-type-support-map-h

Conversation

@christophebedard
Copy link
Copy Markdown
Member

Description

It was deprecated almost 5 years ago in #81.

I don't think it's used anymore, but CI will confirm that.

Is this user-facing behavior change?

not really, and this has been deprecated with a warning for almost 5 years

Did you use Generative AI?

Additional Information

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard self-assigned this Aug 13, 2025
Copy link
Copy Markdown
Contributor

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

good eye 👁️ lgtm

@christophebedard
Copy link
Copy Markdown
Member Author

christophebedard commented Aug 14, 2025

Pulls: #167
Gist: https://gist.githubusercontent.com/christophebedard/65630a979e9eea5fde2ae873c4b86ed5/raw/cd14cc85669525816e75f7aa2de326aa5724f244/ros2.repos
BUILD args:
TEST args:
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16709

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@christophebedard
Copy link
Copy Markdown
Member Author

All test failures are known and unrelated.

@christophebedard christophebedard merged commit 2fa154b into rolling Aug 15, 2025
3 checks passed
@christophebedard christophebedard deleted the christophebedard/remove-deprecated-type-support-map-h branch August 15, 2025 15:08
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