vortex-auv icon indicating copy to clipboard operation
vortex-auv copied to clipboard

Add C++ and CMake pre-commit hooks

Open kluge7 opened this issue 1 year ago • 3 comments

kluge7 avatar Apr 06 '25 19:04 kluge7

Need to remove vortex-auv from here before merging: https://github.com/organizations/vortexntnu/settings/installations/57321089

kluge7 avatar Apr 06 '25 19:04 kluge7

Codecov Report

:x: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 24.01%. Comparing base (38af952) to head (c6915a7).

Files with missing lines Patch % Lines
mission/FSM/docking/src/docking.cpp 0.00% 5 Missing :warning:
...lude/ekf_pose_filtering/ekf_pose_filtering_ros.hpp 0.00% 1 Missing :warning:
...lude/pose_action_server/pose_action_server_ros.hpp 0.00% 1 Missing :warning:
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #568   +/-   ##
=======================================
  Coverage   24.01%   24.01%           
=======================================
  Files          38       38           
  Lines        2286     2286           
  Branches      612      612           
=======================================
  Hits          549      549           
  Misses       1588     1588           
  Partials      149      149           
Flag Coverage Δ
unittests 24.01% <0.00%> (ø)

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

Files with missing lines Coverage Δ
...s_controller/src/dp_adapt_backs_controller_ros.cpp 0.81% <ø> (ø)
...ntroller_dp/include/pid_controller_dp/typedefs.hpp 0.00% <ø> (ø)
...euler/include/pid_controller_dp_euler/typedefs.hpp 0.00% <ø> (ø)
...los_guidance/include/los_guidance/los_guidance.hpp 100.00% <ø> (ø)
...lude/thrust_allocator_auv/thrust_allocator_ros.hpp 0.00% <ø> (ø)
...de/thrust_allocator_auv/thrust_allocator_utils.hpp 0.00% <ø> (ø)
...er_interface_auv/thruster_interface_auv_driver.hpp 0.00% <ø> (ø)
...uster_interface_auv/thruster_interface_auv_ros.hpp 0.00% <ø> (ø)
...nterface_auv/src/thruster_interface_auv_driver.cpp 0.00% <ø> (ø)
...lude/ekf_pose_filtering/ekf_pose_filtering_ros.hpp 0.00% <0.00%> (ø)
... and 2 more
:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Apr 06 '25 19:04 codecov[bot]

Need to remove vortex-auv from here before merging: https://github.com/organizations/vortexntnu/settings/installations/57321089

Don't need to do this, made a separate config file for the local pre-commit hooks. To run locally:

pre-commit run --all-files -c .pre-commit-config-local.yaml

Could also be an idea to make a script which runs both

kluge7 avatar Apr 09 '25 20:04 kluge7