Add C++ and CMake pre-commit hooks
Need to remove vortex-auv from here before merging: https://github.com/organizations/vortexntnu/settings/installations/57321089
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).
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.
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