[CI] Fix creating list of artifacts for CodeCov. #1675
Conversation
mtl1979
commented
Feb 19, 2024
- Remove trailing comma from list of CodeCov artifacts to avoid warning and subsequent error state.
| run: | | ||
| python3 -u -m pip install --user gcovr==5.0 | ||
| python3 -m gcovr -j 3 --verbose \ | ||
| python3 -m gcovr ${{ matrix.build-dir || '' }} -j 3 --verbose \ |
There was a problem hiding this comment.
Why is this needed? I thought that is the purpose of the --root arg below.
There was a problem hiding this comment.
For out-of-source builds, the build/binary directory needs to be sub-directory of the source directory for --root argument to work, it's limitation of filtering in gcovr... However it is possible to pass binary directory separately without that restriction.
There was a problem hiding this comment.
Why was out of source builds previously working fine?
There was a problem hiding this comment.
I don't know why but recently I noticed that the size of coverity artifact dropped from few tens of kilobytes on normal builds to about 300 bytes on OSB builds. It was pretty clear hint that no coverity data was found or collected.