Skip to content

Implement DDS Participants#126

Draft
irenebm wants to merge 9 commits intofeature/discovery_server_participantfrom
feature/dds_participants
Draft

Implement DDS Participants#126
irenebm wants to merge 9 commits intofeature/discovery_server_participantfrom
feature/dds_participants

Conversation

@irenebm
Copy link
Contributor

@irenebm irenebm commented Aug 28, 2024

@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2024

Codecov Report

Attention: Patch coverage is 18.56764% with 307 lines in your changes missing coverage. Please review.

Project coverage is 37.17%. Comparing base (a62ed36) to head (2644c9b).

Files with missing lines Patch % Lines
...cpp/participant/dds/DiscoveryServerParticipant.cpp 18.24% 99 Missing and 13 partials ⚠️
...rc/cpp/participant/dds/InitialPeersParticipant.cpp 17.64% 100 Missing and 12 partials ⚠️
...ants/src/cpp/participant/dds/CommonParticipant.cpp 16.98% 34 Missing and 10 partials ⚠️
...ants/src/cpp/participant/dds/SimpleParticipant.cpp 24.39% 27 Missing and 4 partials ⚠️
...cipants/src/cpp/participant/dds/XmlParticipant.cpp 20.00% 4 Missing and 4 partials ⚠️
Additional details and impacted files
@@                           Coverage Diff                            @@
##           feature/discovery_server_participant     #126      +/-   ##
========================================================================
- Coverage                                 37.83%   37.17%   -0.67%     
========================================================================
  Files                                       153      156       +3     
  Lines                                      6977     7333     +356     
  Branches                                   2788     2960     +172     
========================================================================
+ Hits                                       2640     2726      +86     
- Misses                                     2918     3144     +226     
- Partials                                   1419     1463      +44     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@irenebm irenebm force-pushed the feature/dds_participants branch from fb57143 to c09780e Compare September 2, 2024 10:24
@irenebm irenebm force-pushed the feature/dds_participants branch from c09780e to cd09de4 Compare September 2, 2024 10:29
@irenebm irenebm force-pushed the feature/dds_participants branch from cd09de4 to 2644c9b Compare September 2, 2024 14:53
@irenebm irenebm changed the base branch from main to feature/discovery_server_participant September 2, 2024 14:54
@irenebm irenebm force-pushed the feature/discovery_server_participant branch from a62ed36 to 72e78d0 Compare September 17, 2024 15:26
@irenebm irenebm force-pushed the feature/dds_participants branch from 2644c9b to 39e902e Compare September 20, 2024 09:23
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
Signed-off-by: Irene Bandera <irenebandera@eprosima.com>
…reate_writer and create_reader

Signed-off-by: Irene Bandera <irenebandera@eprosima.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.

2 participants