fix: pre-commit settings (#300)

* fix: pre-commit settings Signed-off-by: Azumi Suzuki <azumi.suzuki@tier4.jp> * ci(pre-commit): autofix --------- Signed-off-by: Azumi Suzuki <azumi.suzuki@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
tier4 · Mar 15, 2023 · dd46088 · dd46088
1 parent a5323ad
commit dd46088
Show file tree

Hide file tree

Showing 17 changed files with 15 additions and 30 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.4.0
     hooks:
       - id: check-json
       - id: check-merge-conflict
@@ -19,23 +19,23 @@ repos:
         args: [--markdown-linebreak-ext=md]
 
   - repo: https://github.com/igorshubovych/markdownlint-cli
-    rev: v0.31.1
+    rev: v0.33.0
     hooks:
       - id: markdownlint
         args: [-c, .markdownlint.yaml, --fix]
 
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: v2.7.1
+    rev: v3.0.0-alpha.4
     hooks:
       - id: prettier
 
   - repo: https://github.com/adrienverge/yamllint
-    rev: v1.26.3
+    rev: v1.29.0
     hooks:
       - id: yamllint
 
   - repo: https://github.com/tier4/pre-commit-hooks-ros
-    rev: v0.7.1
+    rev: v0.8.0
     hooks:
       - id: flake8-ros
       - id: prettier-xacro
@@ -45,35 +45,35 @@ repos:
       - id: sort-package-xml
 
   - repo: https://github.com/shellcheck-py/shellcheck-py
-    rev: v0.8.0.4
+    rev: v0.9.0.2
     hooks:
       - id: shellcheck
 
   - repo: https://github.com/scop/pre-commit-shfmt
-    rev: v3.5.1-1
+    rev: v3.6.0-1
     hooks:
       - id: shfmt
         args: [-w, -s, -i=4]
 
   - repo: https://github.com/pycqa/isort
-    rev: 5.10.1
+    rev: 5.12.0
     hooks:
       - id: isort
 
   - repo: https://github.com/psf/black
-    rev: 22.6.0
+    rev: 23.1.0
     hooks:
       - id: black
         args: [--line-length=100]
 
   - repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: v14.0.6
+    rev: v15.0.7
     hooks:
       - id: clang-format
         types_or: [c++, c, cuda]
 
   - repo: https://github.com/cpplint/cpplint
-    rev: 1.6.0
+    rev: 1.6.1
     hooks:
       - id: cpplint
         args: [--quiet]

diff --git a/common/autoware_testing/autoware_testing/smoke_test.py b/common/autoware_testing/autoware_testing/smoke_test.py
@@ -52,7 +52,6 @@ def resolve_node(context, *args, **kwargs):
 
 @pytest.mark.launch_test
 def generate_test_description():
-
     arg_package = DeclareLaunchArgument(
         "arg_package", default_value=["default"], description="Package containing tested executable"
     )

diff --git a/common/tier4_debug_tools/package.xml b/common/tier4_debug_tools/package.xml
@@ -21,8 +21,8 @@
   <depend>tier4_debug_msgs</depend>
 
   <exec_depend>launch_ros</exec_depend>
-  <exec_depend>rclpy</exec_depend>
   <exec_depend condition="$ROS_PYTHON_VERSION == 3">python3-rtree</exec_depend>
+  <exec_depend>rclpy</exec_depend>
   <test_depend>ament_lint_auto</test_depend>
   <test_depend>autoware_lint_common</test_depend>
 

diff --git a/common/web_controller/www/web_controller/js/menu.js b/common/web_controller/www/web_controller/js/menu.js
@@ -18,7 +18,7 @@ $(function () {
         direction: "horizontal",
       },
       1000,
-      callback(tabName)
+      callback(tabName),
     );
     lastTabName = tabName;
   }
@@ -31,7 +31,7 @@ $(function () {
         {
           direction: "horizontal",
         },
-        1000
+        1000,
       );
     }, 1100);
   }

diff --git a/control/external_cmd_selector/launch/external_cmd_selector.launch.py b/control/external_cmd_selector/launch/external_cmd_selector.launch.py
@@ -24,7 +24,6 @@ def _create_mapping_tuple(name):
 
 
 def generate_launch_description():
-
     arguments = [
         # component
         DeclareLaunchArgument("use_intra_process"),

diff --git a/...ion_launch/launch/obstacle_segmentation/ground_segmentation/ground_segmentation.launch.py b/...ion_launch/launch/obstacle_segmentation/ground_segmentation/ground_segmentation.launch.py
@@ -248,7 +248,6 @@ def create_common_pipeline(self, input_topic, output_topic):
         return components
 
     def create_single_frame_obstacle_segmentation_components(self, input_topic, output_topic):
-
         additional_lidars = self.ground_segmentation_param["additional_lidars"]
         use_ransac = bool(self.ground_segmentation_param["ransac_input_topics"])
         use_additional = bool(additional_lidars)
@@ -425,7 +424,6 @@ def create_single_frame_outlier_filter_components(input_topic, output_topic):
 
     @staticmethod
     def get_additional_lidars_concatenated_component(input_topics, output_topic):
-
         return ComposableNode(
             package="pointcloud_preprocessor",
             plugin="pointcloud_preprocessor::PointCloudConcatenateDataSynchronizerComponent",
@@ -505,7 +503,6 @@ def launch_setup(context, *args, **kwargs):
 
 
 def generate_launch_description():
-
     launch_arguments = []
 
     def add_launch_arg(name: str, default_value=None):

diff --git a/...aunch/launch/scenario_planning/lane_driving/behavior_planning/behavior_planning.launch.py b/...aunch/launch/scenario_planning/lane_driving/behavior_planning/behavior_planning.launch.py
@@ -34,7 +34,6 @@
 
 
 def launch_setup(context, *args, **kwargs):
-
     # vehicle information parameter
     vehicle_info_param_path = LaunchConfiguration("vehicle_info_param_file").perform(context)
     with open(vehicle_info_param_path, "r") as f:

diff --git a/launch/tier4_planning_launch/launch/scenario_planning/parking.launch.py b/launch/tier4_planning_launch/launch/scenario_planning/parking.launch.py
@@ -31,7 +31,6 @@
 
 
 def launch_setup(context, *args, **kwargs):
-
     vehicle_info_param_path = LaunchConfiguration("vehicle_info_param_file").perform(context)
     with open(vehicle_info_param_path, "r") as f:
         vehicle_info_param = yaml.safe_load(f)["/**"]["ros__parameters"]

diff --git a/map/map_loader/test/lanelet2_map_loader_launch.test.py b/map/map_loader/test/lanelet2_map_loader_launch.test.py
@@ -27,7 +27,6 @@
 
 @pytest.mark.launch_test
 def generate_test_description():
-
     lanelet2_map_path = os.path.join(
         get_package_share_directory("map_loader"), "test/data/test_map.osm"
     )

diff --git a/planning/obstacle_cruise_planner/scripts/trajectory_visualizer.py b/planning/obstacle_cruise_planner/scripts/trajectory_visualizer.py
@@ -36,7 +36,6 @@
 
 class TrajectoryVisualizer(Node):
     def __init__(self):
-
         super().__init__("trajectory_visualizer")
 
         self.fig = plt.figure()

diff --git a/planning/planning_debug_tools/scripts/trajectory_visualizer.py b/planning/planning_debug_tools/scripts/trajectory_visualizer.py
@@ -68,7 +68,6 @@
 
 class TrajectoryVisualizer(Node):
     def __init__(self):
-
         super().__init__("trajectory_visualizer")
 
         self.fig = plt.figure()

diff --git a/system/system_monitor/launch/system_monitor.launch.py b/system/system_monitor/launch/system_monitor.launch.py
@@ -25,7 +25,6 @@
 
 
 def launch_setup(context, *args, **kwargs):
-
     with open(LaunchConfiguration("cpu_monitor_config_file").perform(context), "r") as f:
         cpu_monitor_config = yaml.safe_load(f)["/**"]["ros__parameters"]
     cpu_monitor = ComposableNode(

diff --git a/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/calc_utils.py b/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/calc_utils.py
@@ -69,7 +69,6 @@ def lowpass_filter_scipy(x, sample_rate, fp, fs, g_pass, g_stop):
 
     @staticmethod
     def create_2d_map(x, y, data, color_factor, x_index_list, x_thresh, y_index_list, y_thresh):
-
         if x.shape != y.shape or y.shape != data.shape:
             print("Error: the shape of x, y, data must be same")
             sys.exit()

diff --git a/...cel_brake_map_calibrator/accel_brake_map_calibrator/scripts/new_accel_brake_map_server.py b/...cel_brake_map_calibrator/accel_brake_map_calibrator/scripts/new_accel_brake_map_server.py
@@ -224,7 +224,6 @@ def view_pedal_accel_graph(
         calibrated_pedal_list,
         calibrated_acc_list,
     ):
-
         fig = plotter.subplot_more(subplot_num)
 
         # calibrated map

diff --git a/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/plotter.py b/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/plotter.py
@@ -102,7 +102,7 @@ def imshow(self, data, left, right, ws, bottom, top, hs, num_data_type="float"):
             origin="lower",
         )
         ys, xs = np.meshgrid(width_range, height_range, indexing="ij")
-        for (x, y, val) in zip(xs.flatten(), ys.flatten(), data.flatten()):
+        for x, y, val in zip(xs.flatten(), ys.flatten(), data.flatten()):
             self.plot_text(x, y, val, num_data_type)
         plt.colorbar()
 

diff --git a/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/view_plot.py b/vehicle/accel_brake_map_calibrator/accel_brake_map_calibrator/scripts/view_plot.py
@@ -149,7 +149,6 @@ def view_pedal_accel_graph(
         calibrated_acc_list,
         scatter_only,
     ):
-
         fig = plotter.subplot_more(subplot_num)
 
         if not scatter_only:

diff --git a/vehicle/external_cmd_converter/launch/external_cmd_converter.launch.py b/vehicle/external_cmd_converter/launch/external_cmd_converter.launch.py
@@ -25,7 +25,6 @@ def _create_mapping_tuple(name):
 
 
 def generate_launch_description():
-
     arguments = [
         # component
         DeclareLaunchArgument("use_intra_process"),