Hi all,

I wanted to get a wider feedback on adding a code coverage CI for C and
Python. Prototype PR Link: https://github.com/numpy/numpy/pull/30911

*Why add this CI:*
- diff-cover provides a way to see coverage only on the changed files,
which is good for big PRs to understand the UT compared to the main branch.
- With the new GH actions-upload (v7), we can get a clickable link to the
report in a single file. For now the PR implements a MD file format as HTML
file leads to ~250MB file

*Why not to add this CI:*
- Today code coverage and dead code can be calculated locally by following
this: https://numpy.org/doc/stable/dev/index.html#test-coverage. So there
is not much value in running this for every PR

Please let us know your thoughts on having this new CI action.

Thanks,
Ganesh
(https://github.com/ganesh-k13)
_______________________________________________
NumPy-Discussion mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3//lists/numpy-discussion.python.org
Member address: [email protected]

Reply via email to