Code Coverage for C and Python

--===============3930402550589424682==
Content-Type: multipart/alternative; boundary="00000000000024fee5064cbea667"

--00000000000024fee5064cbea667
Content-Type: text/plain; charset="UTF-8"

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)

--00000000000024fee5064cbea667
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Hi all,</div><div><br></div><div>I wanted to get a wi=
der feedback on adding a code coverage CI for C and Python. Prototype PR Li=
nk:=C2=A0<a href=3D"https://github.com/numpy/numpy/pull/30911">https://gith=
ub.com/numpy/numpy/pull/30911</a></div><div><br></div><div><b>Why add this =
CI:</b></div><div>- diff-cover provides a way to see coverage only on the c=
hanged files, which is good for big PRs to understand=C2=A0the UT compared =
to the main branch.</div><div>- With the new GH actions-upload (v7), we can=
 get a clickable link to the report in a single file. For now the PR implem=
ents=C2=A0a MD file format as HTML file leads to ~250MB file</div><div><br>=
</div><div><b>Why not to add this CI:</b></div><div>- Today code coverage a=
nd dead code can be calculated locally by following this:=C2=A0<a href=3D"h=
ttps://numpy.org/doc/stable/dev/index.html#test-coverage">https://numpy.org=
/doc/stable/dev/index.html#test-coverage</a>. So there is not much value in=
 running this for every PR</div><div><br></div><div>Please let us know your=
 thoughts on having this new CI action.</div><div><br></div><div>Thanks,</d=
iv><div>Ganesh</div><div>(<a href=3D"https://github.com/ganesh-k13">https:/=
/github.com/ganesh-k13</a>)</div></div>

--00000000000024fee5064cbea667--

--===============3930402550589424682==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
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]

--===============3930402550589424682==--