Skip to content

Bump version of pre-commit hooks #1

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

Draft
wants to merge 27 commits into
base: ros2-master
Choose a base branch
from
Draft

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Apr 1, 2025

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.

@ViktorCVS ViktorCVS force-pushed the ros2-master branch 2 times, most recently from 1a3a441 to fc07bd3 Compare April 25, 2025 18:22
Add 3 unit tests for saturation feature.
Add 7 unit tests for saturation and anti-windup feature.
Add new parameters to the existing unit tests in the package.
ViktorCVS and others added 21 commits April 28, 2025 10:24
The i_bounds and u_bounds conditions were combined using an 'or'
operator. I have updated them to use separate if else statements.
Delete comments related to dynamic reconfigure and ROS, and update several
other comments.
The i_bounds and u_bounds conditions were combined using an 'or'
operator. I have updated them to use separate if else statements.
This commit adds a helper function for comparing values to zero, improving
the readability of the code.
@ViktorCVS ViktorCVS force-pushed the ros2-master branch 3 times, most recently from 64ce2d6 to 3058121 Compare May 5, 2025 16:44
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