Skip to content

Commit

Permalink
Suppress CI coverage for trouble builds
Browse files Browse the repository at this point in the history
  • Loading branch information
rollbear committed Nov 3, 2023
1 parent 3c15832 commit b8444ef
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -356,6 +356,7 @@ jobs:
cxx_standard: 14,
cxx_asan: false,
libcxx: true,
no_coverage: true,
}
# Clang-3.6
- {
Expand Down Expand Up @@ -393,6 +394,7 @@ jobs:
cxx_standard: 14,
cxx_asan: false,
libcxx: true,
no_coverage: true
}
# Clang-3.7
- {
Expand Down Expand Up @@ -504,6 +506,7 @@ jobs:
cxx_standard: 14,
cxx_asan: true,
libcxx: true,
no_coverage: true
}
# Clang-4.0
- {
Expand Down Expand Up @@ -532,6 +535,7 @@ jobs:
cxx_standard: 11,
cxx_asan: false,
libcxx: true,
no_coverage: true
}
- {
name: "Linux Clang 4.0 C++14 / libc++",
Expand Down Expand Up @@ -1572,8 +1576,9 @@ jobs:
then
cxx_flags="${cxx_flags} -fno-omit-frame-pointer"
fi
echo ${{ matrix.config.name }} | grep -q -e '^Windows' || cxx_flags="${cxx_flags} --coverage"
[ -n "${{ matrix.config.no_coverage }}" ] || {
echo ${{ matrix.config.name }} | grep -q -e '^Windows' || cxx_flags="${cxx_flags} --coverage"
}
cmake \
-S . \
-B build \
Expand All @@ -1600,14 +1605,14 @@ jobs:
./build/test/self_test
- name: Collect coverage
if: (!startsWith(matrix.config.name, 'windows'))
if: (!startsWith(matrix.config.name, 'windows') && !matrix.config.no_coverage)
run: |
COV=`echo "${{ matrix.config.name }}}" | grep -q -e "Linux.*Clang" && echo "llvm-cov gcov"|| echo gcov`
${COV} -abcfup `find build/test -name "*.gcno"`
ls *.gcov| grep -v 'include#trompeloeil'|xargs rm
- name: Upload coverage
if: (!startsWith(matrix.config.name, 'windows')) && github.repository == 'rollbear/trompeloeil'
if: (!startsWith(matrix.config.name, 'windows') && !matrix.config.no_coverage) && github.repository == 'rollbear/trompeloeil'
uses: codecov/codecov-action@v3
with:
gcov_include: "include/trompeloeil"
Expand Down

0 comments on commit b8444ef

Please sign in to comment.