Skip to content

Add precommit workflow#176

Closed
destogl wants to merge 1 commit intoros-controls:masterfrom
destogl:precommit-workflow
Closed

Add precommit workflow#176
destogl wants to merge 1 commit intoros-controls:masterfrom
destogl:precommit-workflow

Conversation

@destogl
Copy link
Copy Markdown
Member

@destogl destogl commented Apr 24, 2021

@destogl destogl self-assigned this Apr 24, 2021
@destogl destogl requested review from Karsten1987 and bmagyar April 24, 2021 10:20
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #176 (0facc9b) into master (7187083) will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   35.28%   35.35%   +0.06%     
==========================================
  Files         156       35     -121     
  Lines       13536     2942   -10594     
  Branches     8772     1858    -6914     
==========================================
- Hits         4776     1040    -3736     
+ Misses        792      236     -556     
+ Partials     7968     1666    -6302     
Flag Coverage Δ
unittests 35.35% <ø> (+0.06%) ⬆️

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

Impacted Files Coverage Δ
...ller/test/test_load_forward_command_controller.cpp
...ollers/test/test_joint_group_effort_controller.cpp
..._state_broadcaster/src/joint_state_broadcaster.cpp
..._broadcaster/test/test_joint_state_broadcaster.cpp
...test/test_load_joint_group_position_controller.cpp
...include/joint_trajectory_controller/tolerances.hpp
...lers/test/test_joint_group_position_controller.cpp
...ontroller/test/test_forward_command_controller.cpp
..._state_broadcaster/src/joint_state_broadcaster.cpp
...lers/test/test_joint_group_velocity_controller.cpp
... and 181 more

@destogl destogl closed this Aug 7, 2021
@destogl destogl deleted the precommit-workflow branch September 8, 2021 18:05
gwalck pushed a commit to b-robotized-forks/ros2_controllers that referenced this pull request Jun 7, 2023
* Add ros2 control cli
* Add README
* Use direct args instead of strategy
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.

2 participants