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

Bump version of pre-commit hooks (backport #1079) #1082

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Mar 18, 2024

This pull request contains auto-updated files of the pre-commit config. @ros-controls/ros2-maintainers please run the pre-commit workflow manually on the branch auto-update- before merging.


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

Copy link

codecov bot commented Mar 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.12%. Comparing base (468e030) to head (8a52ec7).

Additional details and impacted files
@@            Coverage Diff             @@
##           humble    #1082      +/-   ##
==========================================
- Coverage   72.15%   72.12%   -0.03%     
==========================================
  Files          40       40              
  Lines        3555     3555              
  Branches     1787     1788       +1     
==========================================
- Hits         2565     2564       -1     
  Misses        679      679              
- Partials      311      312       +1     
Flag Coverage Δ
unittests 72.12% <ø> (-0.03%) ⬇️

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

see 1 file with indirect coverage changes

@destogl destogl merged commit 333beb0 into humble Mar 18, 2024
12 checks passed
@destogl destogl deleted the mergify/bp/humble/pr-1079 branch March 18, 2024 13:17
henrygerardmoore pushed a commit to henrygerardmoore/ros2_controllers that referenced this pull request Jul 19, 2024
…ntrols#1082)

* added parameterized test for the controller's update rate check
* added warning for the update rate to be a perfect factor
* fix the issue with the period when the controller update rate is higher than the controller manager
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