Skip to content

Add test for defered service callback signature - #478

Merged
ivanpauno merged 6 commits into
masterfrom
ivanpauno/support-defered-response-in-services
Jul 22, 2021
Merged

ivanpauno merged 6 commits into
masterfrom
ivanpauno/support-defered-response-in-services

Conversation

@ivanpauno

Copy link
Copy Markdown
Member

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@ivanpauno ivanpauno added the enhancement New feature or request label Jul 13, 2021
@ivanpauno
ivanpauno requested a review from wjwwood July 13, 2021 21:40
@ivanpauno ivanpauno self-assigned this Jul 13, 2021
Comment thread test_rclcpp/test/test_services_server.cpp Outdated
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@ivanpauno

Copy link
Copy Markdown
Member Author

CI: ros2/rclcpp#1709 (comment)

@ivanpauno
ivanpauno requested a review from wjwwood July 16, 2021 21:48
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@ivanpauno

Copy link
Copy Markdown
Member Author

Last commit should've fixed the CI issues

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@ivanpauno
ivanpauno requested a review from wjwwood July 20, 2021 21:45
@ivanpauno

Copy link
Copy Markdown
Member Author

Finally got windows CI passing: ros2/rclcpp#1709 (comment).

@ivanpauno

Copy link
Copy Markdown
Member Author

@ros-pull-request-builder retest this please

@ivanpauno
ivanpauno merged commit 4d7324c into master Jul 22, 2021
@delete-merged-branch
delete-merged-branch Bot deleted the ivanpauno/support-defered-response-in-services branch July 22, 2021 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants