Skip to content

Conversation

@ivanpauno
Copy link
Collaborator

Depends on #14.

This is adding tests for subscription events.

@ivanpauno ivanpauno added the enhancement New feature or request label Aug 20, 2020
@ivanpauno ivanpauno requested a review from jacobperron August 20, 2020 19:17
@ivanpauno ivanpauno self-assigned this Aug 20, 2020
@ivanpauno ivanpauno force-pushed the ivanpauno/add-subscription-requested-qos-incompatible-event branch from b39a75a to 1497e24 Compare August 24, 2020 17:00
Base automatically changed from ivanpauno/add-subscription-requested-qos-incompatible-event to feature/events August 24, 2020 21:36
@ivanpauno ivanpauno force-pushed the ivanpauno/test-subscription-events branch from 98c3487 to 63435b7 Compare August 25, 2020 14:25
@ivanpauno ivanpauno marked this pull request as ready for review August 25, 2020 14:25
Copy link

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Besides the left-over print statements that can be removed, this LGTM

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
@ivanpauno ivanpauno merged commit 46a5cdc into feature/events Aug 26, 2020
@ivanpauno ivanpauno deleted the ivanpauno/test-subscription-events branch August 26, 2020 14:20
ivanpauno added a commit that referenced this pull request Aug 31, 2020
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
ivanpauno added a commit that referenced this pull request May 17, 2021
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 14, 2022
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 25, 2022
Signed-off-by: Ivan Santiago Paunovic <[email protected]>
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.

3 participants