Update the github workflow to build on all supported platforms. (#547)

* Update the github workflow to build on all supported platforms. And to use non-outdated versions of the actions. Signed-off-by: Chris Lalancette <[email protected]>
ros-drivers · Oct 30, 2024 · 8a0a663 · 8a0a663
1 parent 1d9cce9
commit 8a0a663
Showing 1 changed file with 9 additions and 20 deletions.
diff --git a/.github/workflows/basic-build-ci.yaml b/.github/workflows/basic-build-ci.yaml
@@ -2,32 +2,21 @@ name: Basic Build Workflow
 
 on:
   - pull_request
-  - push
 
 jobs:
-  build-ros2-latest:
-    runs-on: ubuntu-22.04
+  build:
+    runs-on: ubuntu-latest
     strategy:
       matrix:
-        image:
-          - rostooling/setup-ros-docker:ubuntu-jammy-ros-iron-ros-base-latest
-          - rostooling/setup-ros-docker:ubuntu-jammy-ros-rolling-ros-base-latest
-        ros-version:
-          - iron
-          - rolling
-        exclude:
-          - image: rostooling/setup-ros-docker:ubuntu-jammy-ros-iron-ros-base-latest
-            ros-version: rolling
-          - image: rostooling/setup-ros-docker:ubuntu-jammy-ros-rolling-ros-base-latest
-            ros-version: iron
+        rosdistro: [humble, iron, jazzy, rolling]
       fail-fast: false
     container:
-      image: ${{ matrix.image }}
+      image: ros:${{ matrix.rosdistro }}-ros-base
     steps:
       - name: Build Environment
-        uses: ros-tooling/setup-ros@v0.6
+        uses: ros-tooling/setup-ros@v0.7
         with:
-          required-ros-distributions: ${{ matrix.ros-version }}
+          required-ros-distributions: ${{ matrix.rosdistro }}
       - name: Run Tests
         uses: ros-tooling/[email protected]
         with:
@@ -37,10 +26,10 @@ jobs:
             velodyne_laserscan
             velodyne_msgs
             velodyne_pointcloud
-          target-ros2-distro: ${{ matrix.ros-version }}
+          target-ros2-distro: ${{ matrix.rosdistro }}
       - name: Upload logs
-        uses: actions/upload-artifact@v1
+        uses: actions/upload-artifact@v4
         with:
-          name: colcon-logs
+          name: colcon-logs-${{ matrix.rosdistro }}
           path: ros_ws/log
         if: always()