-
Notifications
You must be signed in to change notification settings - Fork 49
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
Added rmw_client, rmw_destroy_client and rmw_service_server_is_available tests #140
Conversation
Related with this PR ros2/rmw_fastrtps#447 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's one overlap with #138 we have to sort out.
91054b3
to
abfeb84
Compare
Alright, PR rebased on top of #138. |
@ros-pull-request-builder retest this please. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@ros-pull-request-builder retest this please. |
…ble tests Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
abfeb84
to
a4e2195
Compare
Depends on ros2/rosidl_typesupport_connext#60 to be released and rolling to be upped |
@ros-pull-request-builder retest this please. |
3 similar comments
@ros-pull-request-builder retest this please. |
@ros-pull-request-builder retest this please. |
@ros-pull-request-builder retest this please. |
It passed ! Finally all green. Merging! |
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Testing:
Related to this PR
Signed-off-by: ahcorde ahcorde@gmail.com