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

[diff_drive] Remove unused parameter and add simple validation #abi-breaking (backport #958) #1056

Merged

Merge branch 'iron' into mergify/bp/iron/pr-958

127b9e1
Select commit
Loading
Failed to load commit list.
Merged

[diff_drive] Remove unused parameter and add simple validation #abi-breaking (backport #958) #1056

Merge branch 'iron' into mergify/bp/iron/pr-958
127b9e1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 29, 2024 in 0s

47.12% (-0.35%) compared to 4941fee

View this Pull Request on Codecov

47.12% (-0.35%) compared to 4941fee

Details

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.12%. Comparing base (4941fee) to head (127b9e1).

Additional details and impacted files
@@            Coverage Diff             @@
##             iron    #1056      +/-   ##
==========================================
- Coverage   47.46%   47.12%   -0.35%     
==========================================
  Files          40       40              
  Lines        3685     3682       -3     
  Branches     1758     1756       -2     
==========================================
- Hits         1749     1735      -14     
- Misses        722      735      +13     
+ Partials     1214     1212       -2     
Files Coverage Δ
...de/diff_drive_controller/diff_drive_controller.hpp 100.00% <ø> (ø)
...iff_drive_controller/src/diff_drive_controller.cpp 41.49% <100.00%> (-2.95%) ⬇️

... and 1 file with indirect coverage changes