Include compiler in CI artifact names, fix coverage

Signed-off-by: Evan Flynn <[email protected]>
ros-drivers · Feb 9, 2025 · a0afde3 · a0afde3
1 parent c68f69e
commit a0afde3
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/.github/workflows/build_test.yml b/.github/workflows/build_test.yml
@@ -118,7 +118,14 @@ jobs:
           package-name: usb_cam
           target-ros2-distro: ${{ matrix.ros_distro }}
           vcs-repo-file-url: ""
-          extra-cmake-args: -DCMAKE_C_COMPILER=${{ matrix.cc_compiler }} -DCMAKE_CXX_COMPILER=${{ matrix.cxx_compiler }}
+          extra-cmake-args: -DCMAKE_C_COMPILER=${{ matrix.cc_compiler }} -DCMAKE_CXX_COMPILER=${{ matrix.cxx_compiler }} -DLLVM_ENABLE_RUNTIMES=compiler-rt
+          colcon-defaults: |
+            {
+              "build": {
+                "mixin": ["coverage-gcc"]
+              }
+            }
+          colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/b8436aa16c0bdbc01081b12caa253cbf16e0fb82/index.yaml
       # Compile again, this time enabling integration tests
       - name: Build integration tests
         shell: bash
@@ -132,13 +139,13 @@ jobs:
             renode_portable/renode --disable-gui --version
       - uses: actions/upload-artifact@v4
         with:
-          name: colcon-logs-${{ matrix.ros_distro }}
+          name: colcon-logs-${{ matrix.ros_distro }}-${{ matrix.cxx_compiler }}
           path: ${{ steps.build_and_test_step.outputs.ros-workspace-directory-name }}/log
         if: always()
         continue-on-error: true
       - uses: actions/upload-artifact@v4
         with:
-          name: lcov-logs-${{ matrix.ros_distro }}
+          name: lcov-logs-${{ matrix.ros_distro }}-${{ matrix.cxx_compiler }}
           path: ${{ steps.build_and_test_step.outputs.ros-workspace-directory-name }}/lcov
         if: always()
         continue-on-error: true