Precommit: Use pyproject.toml and fix cppcheck

bit-bots · Aug 3, 2023 · f489220 · f489220
1 parent ce6d14f
commit f489220
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,20 +4,19 @@ repos:
   hooks:
     - id: ruff
       args:
-        - "--line-length=120"
         - "--fix"
         - "--exit-non-zero-on-fix"
-        - "--select=F,E,W,I,N,UP"
 - repo: https://github.com/psf/black
   rev: 23.3.0
   hooks:
     - id: black
-      args:
-        - "--line-length=120"
 - repo: https://github.com/pocc/pre-commit-hooks
   rev: v1.3.5
   hooks:
     - id: clang-format
       args:
         - "-i"
     - id: cppcheck
+      args:
+        - "--suppress=missingInclude"
+        - "--suppress=missingIncludeSystem"
diff --git a/bitbots_head_mover/src/move_head.cpp b/bitbots_head_mover/src/move_head.cpp
@@ -348,8 +348,8 @@ class HeadMover {
   }
 
   std::pair<double, double> get_head_position() {
-    double head_pan;
-    double head_tilt;
+    double head_pan = 0.0;
+    double head_tilt = 0.0;
     for (size_t i = 0; i < current_joint_state_.name.size(); i++) {
       if (current_joint_state_.name[i] == "HeadPan") {
         head_pan = current_joint_state_.position[i];

diff --git a/pyproject.toml b/pyproject.toml
@@ -0,0 +1,6 @@
+[tool.black]
+line-length = 120
+
+[tool.ruff]
+line-length = 120
+select = ["F", "E", "W", "I", "N", "UP"]