Skip to content

Add rmw_count clients,services impl#641

Merged
clalancette merged 3 commits intoros2:rollingfrom
leeminju531:rolling
Sep 29, 2023
Merged

Add rmw_count clients,services impl#641
clalancette merged 3 commits intoros2:rollingfrom
leeminju531:rolling

Conversation

@leeminju531
Copy link
Copy Markdown
Contributor

this PR is for service counts using it's name like topic.
Related to ros2/ros2cli#771

Signed-off-by: leeminju531 dlalswn531@naver.com

@gbiggs
Copy link
Copy Markdown
Member

gbiggs commented Nov 24, 2022

CI:

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

@leeminju531
Copy link
Copy Markdown
Contributor Author

image

@gbiggs friendly ping, could you build again with CI_USE_FASTRTPS_DYNAMIC ?

@gbiggs
Copy link
Copy Markdown
Member

gbiggs commented Nov 27, 2022

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

Copy link
Copy Markdown
Collaborator

@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.

@fujitatomoya
Copy link
Copy Markdown
Collaborator

CC: @MiguelCompany

Copy link
Copy Markdown
Collaborator

@MiguelCompany MiguelCompany left a comment

Choose a reason for hiding this comment

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

LGTM

@methylDragon
Copy link
Copy Markdown
Contributor

methylDragon commented Feb 9, 2023

Signed-off-by: leeminju531 <dlalswn531@naver.com>
Signed-off-by: leeminju531 <dlalswn531@naver.com>
Signed-off-by: leeminju531 <dlalswn531@naver.com>
@clalancette
Copy link
Copy Markdown
Contributor

CI for this is in ros2/rmw_implementation#208 (comment)

@clalancette clalancette merged commit ad4550a into ros2:rolling Sep 29, 2023
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.

6 participants