External/pugixml
8
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix coverage build to include CXX define

Browse Source 
Also it seems like we don't need to restrict coverage build to
Linux/clang?
This commit is contained in:
Arseny Kapoulkine 2021-06-27 19:20:27 -07:00
parent 0de580155d
commit 5cff00e5ea
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/build.yml vendored
View File

@ -26,8 +26,8 @@ jobs:
make test cxxstd=c++98 defines=${{matrix.defines}} config=debug -j2 make test cxxstd=c++98 defines=${{matrix.defines}} config=debug -j2
make test defines=${{matrix.defines}} config=sanitize -j2 make test defines=${{matrix.defines}} config=sanitize -j2
- name: make coverage - name: make coverage
if: ${{matrix.os == 'ubuntu' && matrix.compiler == 'clang++'}}
run: | run: |
export CXX=${{matrix.compiler}}
make test defines=${{matrix.defines}} config=coverage -j2 make test defines=${{matrix.defines}} config=coverage -j2
bash <(curl -s https://codecov.io/bash) -f pugixml.cpp.gcov -X search -t ${{secrets.CODECOV_TOKEN}} -B ${{github.ref}} bash <(curl -s https://codecov.io/bash) -f pugixml.cpp.gcov -X search -t ${{secrets.CODECOV_TOKEN}} -B ${{github.ref}}