Skip to content

Add tests service/client request/response with bad arguments#141

Merged
Lobotuerk merged 7 commits intomasterfrom
lobotuerk/rmw-client-send-request-tests
Sep 29, 2020
Merged

Add tests service/client request/response with bad arguments#141
Lobotuerk merged 7 commits intomasterfrom
lobotuerk/rmw-client-send-request-tests

Conversation

@Lobotuerk
Copy link
Copy Markdown
Contributor

No description provided.

@Lobotuerk Lobotuerk marked this pull request as draft September 25, 2020 15:57
@Lobotuerk Lobotuerk changed the title Add tests rmw_client_send_request with bad arguments Add tests service/client request/response with bad arguments Sep 28, 2020
@Lobotuerk Lobotuerk marked this pull request as ready for review September 28, 2020 19:54
@Lobotuerk Lobotuerk requested a review from hidmic September 28, 2020 21:36
@Lobotuerk Lobotuerk requested a review from hidmic September 28, 2020 22:36
Copy link
Copy Markdown

@hidmic hidmic left a comment

Choose a reason for hiding this comment

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

LGTM with green CI ! Thanks for pushing @Lobotuerk !

Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
@hidmic hidmic force-pushed the lobotuerk/rmw-client-send-request-tests branch from b3a29ba to 8e30085 Compare September 29, 2020 12:43
@hidmic
Copy link
Copy Markdown

hidmic commented Sep 29, 2020

Rebased on top of #138.

CI up to rcl and test_rmw_implementation:

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

@Lobotuerk
Copy link
Copy Markdown
Contributor Author

@ros-pull-request-builder retest this please.

2 similar comments
@Lobotuerk
Copy link
Copy Markdown
Contributor Author

@ros-pull-request-builder retest this please.

@hidmic
Copy link
Copy Markdown

hidmic commented Sep 29, 2020

@ros-pull-request-builder retest this please.

@Lobotuerk
Copy link
Copy Markdown
Contributor Author

Merging after all green CI

@Lobotuerk Lobotuerk merged commit ef60ec6 into master Sep 29, 2020
@delete-merged-branch delete-merged-branch bot deleted the lobotuerk/rmw-client-send-request-tests branch September 29, 2020 16:17
ahcorde pushed a commit that referenced this pull request Oct 9, 2020
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
ahcorde pushed a commit that referenced this pull request Oct 21, 2020
Signed-off-by: lobotuerk <jtlorente@ekumenlabs.com>
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.

3 participants