Skip to content

[diff_drive] Remove non-existing parameter (backport #1119) #1127

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

Merged
merged 1 commit into from
May 9, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 9, 2024

I removed the unused wheels_per_side parameter with #958 but forgot to remove it from the test-yaml printed into the docs.


This is an automatic backport of pull request #1119 done by Mergify.

Copy link

codecov bot commented May 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.79%. Comparing base (2e8b4e4) to head (756f4c8).

Additional details and impacted files
@@            Coverage Diff             @@
##             iron    #1127      +/-   ##
==========================================
+ Coverage   86.44%   86.79%   +0.34%     
==========================================
  Files          86       86              
  Lines        7397     7396       -1     
  Branches      614      613       -1     
==========================================
+ Hits         6394     6419      +25     
+ Misses        773      749      -24     
+ Partials      230      228       -2     
Flag Coverage Δ
unittests 86.79% <ø> (+0.34%) ⬆️

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

see 4 files with indirect coverage changes

@christophfroehlich christophfroehlich merged commit b46eaee into iron May 9, 2024
12 of 13 checks passed
@christophfroehlich christophfroehlich deleted the mergify/bp/iron/pr-1119 branch May 9, 2024 12:00
henrygerardmoore pushed a commit to henrygerardmoore/ros2_controllers that referenced this pull request Jul 19, 2024
# 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