Ci updates (#7)

* Remove foxy CI * Fix downstream workspace * Use ament cmake tests
LeoRover · Nov 2, 2023 · 6d70cb7 · 6d70cb7
1 parent 95f19f3
commit 6d70cb7
Show file tree

Hide file tree

Showing 8 changed files with 28 additions and 78 deletions.
diff --git a/.github/workflows/ci-foxy.yml b/.github/workflows/ci-foxy.yml
diff --git a/.github/workflows/ci-humble.yml b/.github/workflows/ci-humble.yml
@@ -26,28 +26,4 @@ jobs:
       - name: Source tests
         uses: "ros-industrial/industrial_ci@master"
         env:
-          DOWNSTREAM_WORKSPACE: "github:LeoRover/leo_desktop-ros2#humble github:LeoRover/leo_desktop-ros2#humble"
-
-  ament_lint:
-    name: ament_${{ matrix.linter }}
-    runs-on: ubuntu-20.04
-    container:
-      image: ubuntu:jammy
-    strategy:
-      fail-fast: false
-      matrix:
-        linter: [lint_cmake, xmllint]
-    steps:
-      - uses: actions/checkout@v3
-      - uses: ros-tooling/setup-ros@v0.3
-        with:
-          required-ros-distributions: ${{ env.ROS_DISTRO }}
-      - uses: ros-tooling/action-ros-lint@v0.1
-        with:
-          distribution: humble
-          linter: ${{ matrix.linter }}
-          package-name: |
-            leo
-            leo_description
-            leo_msgs
-            leo_teleop
+          DOWNSTREAM_WORKSPACE: "github:LeoRover/leo_desktop-ros2#humble github:LeoRover/leo_robot-ros2#humble"
diff --git a/leo_description/CMakeLists.txt b/leo_description/CMakeLists.txt
@@ -12,6 +12,11 @@ install(
   DESTINATION share/${PROJECT_NAME}
 )
 
+if(BUILD_TESTING)
+  find_package(ament_lint_auto REQUIRED)
+  ament_lint_auto_find_test_dependencies()
+endif()
+
 ament_environment_hooks("${CMAKE_CURRENT_SOURCE_DIR}/hooks/gz_sim_resource_path.dsv")
 
 ament_package()
diff --git a/leo_description/package.xml b/leo_description/package.xml
@@ -20,6 +20,10 @@
   <exec_depend>xacro</exec_depend>
   <exec_depend>robot_state_publisher</exec_depend>
 
+  <test_depend>ament_lint_auto</test_depend>
+  <test_depend>ament_cmake_lint_cmake</test_depend>
+  <test_depend>ament_cmake_xmllint</test_depend>
+
   <export>
     <build_type>ament_cmake</build_type>
   </export>

diff --git a/leo_msgs/CMakeLists.txt b/leo_msgs/CMakeLists.txt
@@ -14,4 +14,9 @@ rosidl_generate_interfaces(${PROJECT_NAME}
   DEPENDENCIES builtin_interfaces
 )
 
+if(BUILD_TESTING)
+  find_package(ament_lint_auto REQUIRED)
+  ament_lint_auto_find_test_dependencies()
+endif()
+
 ament_package()
diff --git a/leo_msgs/package.xml b/leo_msgs/package.xml
@@ -25,6 +25,10 @@
   <build_depend>builtin_interfaces</build_depend>
   <exec_depend>builtin_interfaces</exec_depend>
 
+  <test_depend>ament_lint_auto</test_depend>
+  <test_depend>ament_cmake_lint_cmake</test_depend>
+  <test_depend>ament_cmake_xmllint</test_depend>
+
   <export>
     <build_type>ament_cmake</build_type>
   </export>

diff --git a/leo_teleop/CMakeLists.txt b/leo_teleop/CMakeLists.txt
@@ -10,4 +10,9 @@ install(
   DESTINATION share/${PROJECT_NAME}
 )
 
+if(BUILD_TESTING)
+  find_package(ament_lint_auto REQUIRED)
+  ament_lint_auto_find_test_dependencies()
+endif()
+
 ament_package()
diff --git a/leo_teleop/package.xml b/leo_teleop/package.xml
@@ -21,6 +21,10 @@
   <exec_depend>teleop_twist_joy</exec_depend>
   <exec_depend>teleop_twist_keyboard</exec_depend>
 
+  <test_depend>ament_lint_auto</test_depend>
+  <test_depend>ament_cmake_lint_cmake</test_depend>
+  <test_depend>ament_cmake_xmllint</test_depend>
+
   <export>
     <build_type>ament_cmake</build_type>
   </export>