ROS2 Linting: localization_launch (autowarefoundation#39)

* Add linters to localization_launch package * Add new line * Fix indentation
tkimura4 · Dec 13, 2021 · f6ac5c5 · f6ac5c5
1 parent a847219
commit f6ac5c5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/launch/localization_launch/CMakeLists.txt b/launch/localization_launch/CMakeLists.txt
@@ -2,15 +2,20 @@ cmake_minimum_required(VERSION 3.5)
 project(localization_launch)
 
 if(NOT CMAKE_CXX_STANDARD)
-    set(CMAKE_CXX_STANDARD 14)
+  set(CMAKE_CXX_STANDARD 14)
 endif()
 if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang")
-    add_compile_options(-Wall -Wextra -Wpedantic -Wno-unused-parameter)
+  add_compile_options(-Wall -Wextra -Wpedantic -Wno-unused-parameter)
 endif()
 
 find_package(ament_cmake_auto REQUIRED)
 ament_auto_find_build_dependencies()
 
+if(BUILD_TESTING)
+  find_package(ament_lint_auto REQUIRED)
+  ament_lint_auto_find_test_dependencies()
+endif()
+
 ament_auto_package(INSTALL_TO_SHARE
-        launch
-        )
+  launch
+)
diff --git a/launch/localization_launch/package.xml b/launch/localization_launch/package.xml
@@ -5,16 +5,9 @@
   <version>0.1.0</version>
   <description>The localization_launch package</description>
 
-  <!-- One maintainer tag required, multiple allowed, one person per tag -->
-  <!-- Example:  -->
-  <!-- <maintainer email="jane.doe@example.com">Jane Doe</maintainer> -->
   <maintainer email="yamato.ando@gmail.com">Yamato Ando</maintainer>
 
-
-  <!-- One license tag required, multiple allowed, one license per tag -->
-  <!-- Commonly used license strings: -->
-  <!--   BSD, MIT, Boost Software License, GPLv2, GPLv3, LGPLv2.1, LGPLv3 -->
-  <license>Apache 2</license>
+  <license>Apache License 2.0</license>
 
   <buildtool_depend>ament_cmake_auto</buildtool_depend>
 
@@ -25,8 +18,9 @@
   <depend>pointcloud_preprocessor</depend>
   <depend>topic_tools</depend>
 
+  <test_depend>ament_lint_auto</test_depend>
+  <test_depend>ament_lint_common</test_depend>
 
-  <!-- The export tag contains other, unspecified, tags -->
   <export>
     <build_type>ament_cmake</build_type>
   </export>