[sdarwin]

Adds codecov.io coverage reports

[wchilders-nvidia]

Hm, if we're going to do this, this feels like something we should factor into the primary build and test CI rather than adding yet another CI that only runs a subset of the tested configurations of the primary build & test CI.

[codecov]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

[sdarwin]

factor into the primary build

Updated

[wchilders-nvidia]

Is there some way with gcov to collect the reports and then combine them (similar to what is done with the edgy results)? I'm a little worried about the "magic number" 4 there. We have an initiative we're working on within NVIDIA to expand the number of configs that are tested and exposed within the extensive posture to expand coverage.

[sdarwin]

""" Good idea. Done. Each test leg now saves a gcovr JSON report as an artifact. A single coverage job, like publish-test-results, merges all coverage-* artifacts and uploads once to Codecov. There's no count anywhere anymore, so new configs in the extensive posture are covered without touching the Codecov config. The trade-off is that Codecov no longer shows a per-config breakdown. Coverage steps are all continue-on-error, so they never affect the CI result.