Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Exlusive hw interface rolling #1492

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

firesurfer
Copy link
Contributor

This is the first part for testing #1487

And sorry that I messed up #1491

Copy link

codecov bot commented Apr 19, 2024

Codecov Report

Attention: Patch coverage is 1.14943% with 86 lines in your changes missing coverage. Please review.

Project coverage is 62.20%. Comparing base (35bb5f7) to head (3f49f2d).
Report is 94 commits behind head on master.

Files Patch % Lines
..._components/test_actuator_exclusive_interfaces.cpp 1.44% 68 Missing ⚠️
...ailed_activate/test_controller_failed_activate.cpp 0.00% 13 Missing ⚠️
...ailed_activate/test_controller_failed_activate.hpp 0.00% 5 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1492       +/-   ##
===========================================
- Coverage   75.51%   62.20%   -13.31%     
===========================================
  Files          41       99       +58     
  Lines        3328    11398     +8070     
  Branches     1921     8190     +6269     
===========================================
+ Hits         2513     7090     +4577     
- Misses        421      800      +379     
- Partials      394     3508     +3114     
Flag Coverage Δ
unittests 62.20% <1.14%> (-13.31%) ⬇️

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

Files Coverage Δ
...ailed_activate/test_controller_failed_activate.hpp 0.00% <0.00%> (ø)
...ailed_activate/test_controller_failed_activate.cpp 0.00% <0.00%> (ø)
..._components/test_actuator_exclusive_interfaces.cpp 1.44% <1.44%> (ø)

... and 60 files with indirect coverage changes

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.

1 participant