Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Test lock_free queue #1480

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Test lock_free queue #1480

wants to merge 4 commits into from

Conversation

christophfroehlich
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 56.25000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 84.13%. Comparing base (50ff026) to head (c25740a).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...iff_drive_controller/src/diff_drive_controller.cpp 56.25% 4 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1480      +/-   ##
==========================================
- Coverage   84.19%   84.13%   -0.06%     
==========================================
  Files         123      123              
  Lines       11323    11322       -1     
  Branches      957      959       +2     
==========================================
- Hits         9533     9526       -7     
- Misses       1473     1477       +4     
- Partials      317      319       +2     
Flag Coverage Δ
unittests 84.13% <56.25%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ive_controller/test/test_diff_drive_controller.cpp 94.32% <ø> (ø)
...iff_drive_controller/src/diff_drive_controller.cpp 70.88% <56.25%> (-2.06%) ⬇️

... and 3 files with indirect coverage changes

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant