Skip to content

Commit a0afde3

Browse files
committed
Include compiler in CI artifact names, fix coverage
Signed-off-by: Evan Flynn <[email protected]>
1 parent c68f69e commit a0afde3

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

.github/workflows/build_test.yml

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,14 @@ jobs:
118118
package-name: usb_cam
119119
target-ros2-distro: ${{ matrix.ros_distro }}
120120
vcs-repo-file-url: ""
121-
extra-cmake-args: -DCMAKE_C_COMPILER=${{ matrix.cc_compiler }} -DCMAKE_CXX_COMPILER=${{ matrix.cxx_compiler }}
121+
extra-cmake-args: -DCMAKE_C_COMPILER=${{ matrix.cc_compiler }} -DCMAKE_CXX_COMPILER=${{ matrix.cxx_compiler }} -DLLVM_ENABLE_RUNTIMES=compiler-rt
122+
colcon-defaults: |
123+
{
124+
"build": {
125+
"mixin": ["coverage-gcc"]
126+
}
127+
}
128+
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/b8436aa16c0bdbc01081b12caa253cbf16e0fb82/index.yaml
122129
# Compile again, this time enabling integration tests
123130
- name: Build integration tests
124131
shell: bash
@@ -132,13 +139,13 @@ jobs:
132139
renode_portable/renode --disable-gui --version
133140
- uses: actions/upload-artifact@v4
134141
with:
135-
name: colcon-logs-${{ matrix.ros_distro }}
142+
name: colcon-logs-${{ matrix.ros_distro }}-${{ matrix.cxx_compiler }}
136143
path: ${{ steps.build_and_test_step.outputs.ros-workspace-directory-name }}/log
137144
if: always()
138145
continue-on-error: true
139146
- uses: actions/upload-artifact@v4
140147
with:
141-
name: lcov-logs-${{ matrix.ros_distro }}
148+
name: lcov-logs-${{ matrix.ros_distro }}-${{ matrix.cxx_compiler }}
142149
path: ${{ steps.build_and_test_step.outputs.ros-workspace-directory-name }}/lcov
143150
if: always()
144151
continue-on-error: true

0 commit comments

Comments
 (0)