Skip to content

Add in inconsistent_topic implementation.#1024

Merged
clalancette merged 2 commits intorollingfrom
clalancette/inconsistent-topic-event
Mar 13, 2023
Merged

Add in inconsistent_topic implementation.#1024
clalancette merged 2 commits intorollingfrom
clalancette/inconsistent-topic-event

Conversation

@clalancette
Copy link
Copy Markdown
Contributor

@clalancette clalancette commented Dec 20, 2022

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

Part of ros2/ros2#1361

Depends on ros2/rmw#339

Copy link
Copy Markdown
Member

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

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

LGTM

@clalancette clalancette self-assigned this Jan 5, 2023
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 9fd94f8 to 1ac72a6 Compare January 17, 2023 15:56
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch 2 times, most recently from 992b3d8 to 72cc2d8 Compare February 10, 2023 16:50
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 72cc2d8 to 695300e Compare February 24, 2023 21:07
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch 2 times, most recently from dd94316 to 4e114b7 Compare March 7, 2023 18:56
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette clalancette force-pushed the clalancette/inconsistent-topic-event branch from 4e114b7 to fda537c Compare March 10, 2023 14:52
@clalancette clalancette merged commit a64d867 into rolling Mar 13, 2023
@delete-merged-branch delete-merged-branch bot deleted the clalancette/inconsistent-topic-event branch March 13, 2023 13:32
danthony06 pushed a commit to danthony06/rcl that referenced this pull request Jun 14, 2023
* Add in incompatible type implementation.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
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.

4 participants