Skip to content

Conversation

@jparisu
Copy link
Contributor

@jparisu jparisu commented Aug 24, 2022

This PR requires a Fast DDS PR to be merged: eProsima/Fast-DDS#2943

@jparisu jparisu temporarily deployed to codecov August 24, 2022 12:09 Inactive
@jparisu jparisu temporarily deployed to codecov August 24, 2022 12:09 Inactive
@codecov
Copy link

codecov bot commented Aug 24, 2022

Codecov Report

Attention: Patch coverage is 41.07143% with 33 lines in your changes missing coverage. Please review.

Project coverage is 58.28%. Comparing base (e1447b3) to head (bab8f1a).
Report is 124 commits behind head on main.

Files with missing lines Patch % Lines
...icipant/implementations/rtps/CommonParticipant.cpp 39.02% 6 Missing and 19 partials ⚠️
...ore/src/cpp/writer/implementations/rtps/Writer.cpp 0.00% 0 Missing and 3 partials ⚠️
...ore/src/cpp/reader/implementations/rtps/Reader.cpp 66.66% 0 Missing and 2 partials ⚠️
...ter/implementations/rtps/filter/GuidDataFilter.cpp 60.00% 0 Missing and 2 partials ⚠️
...implementations/rtps/filter/RepeaterDataFilter.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #284      +/-   ##
==========================================
- Coverage   58.45%   58.28%   -0.18%     
==========================================
  Files          99       99              
  Lines        3428     3459      +31     
  Branches     1277     1293      +16     
==========================================
+ Hits         2004     2016      +12     
- Misses        649      653       +4     
- Partials      775      790      +15     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu jparisu force-pushed the feature/local-router-filtering branch from e1bde81 to 6aa7b8c Compare September 6, 2022 10:27
@jparisu jparisu temporarily deployed to codecov September 6, 2022 10:27 Inactive
@jparisu jparisu temporarily deployed to codecov September 6, 2022 10:27 Inactive
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu jparisu temporarily deployed to codecov September 6, 2022 10:39 Inactive
@jparisu jparisu temporarily deployed to codecov September 6, 2022 10:39 Inactive
@rsanchez15 rsanchez15 closed this Jul 21, 2025
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