py-mavros: reconfigure flake8

mavlink · Oct 10, 2024 · 6959261 · 6959261
1 parent 27365df
commit 6959261
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/mavros/CMakeLists.txt b/mavros/CMakeLists.txt
@@ -232,6 +232,9 @@ if(BUILD_TESTING)
   # list(APPEND ament_cmake_cppcheck_ADDITIONAL_INCLUDE_DIRS "${EIGEN3_INCLUDE_DIRS}")
   # set(ament_cmake_cppcheck_LANGUAGE "c++")
 
+  # NOTE(vooon): i prefer black style, which uses double quotes
+  set(ament_cmake_flake8_CONFIG_FILE "./setup.cfg")
+
   # NOTE(vooon): tired to provide more and more macro
   list(APPEND ament_cmake_cppcheck_ADDITIONAL_EXCLUDE "./test/*.cpp")
 

diff --git a/mavros/setup.cfg b/mavros/setup.cfg
@@ -26,5 +26,10 @@ script_dir=$base/lib/mavros
 [install]
 install_scripts=$base/lib/mavros
 
-[yapf]
-blank_line_before_nested_class_or_def = True
+[flake8]
+# NOTE: based on ament_flake8.ini from Jazzy, but extended with Q000
+extend-ignore = B902,C816,D100,D101,D102,D103,D104,D105,D106,D107,D203,D212,D404,I202,Q000
+import-order-style = google
+max-line-length = 99
+show-source = true
+statistics = true