glass icon indicating copy to clipboard operation
glass copied to clipboard

Code coverage

Open ntessore opened this issue 3 years ago • 1 comments

Set up codecov or similar to track coverage of unit tests. This should give us a good push to start adding tests, now that we can have them!

ntessore avatar Feb 05 '23 01:02 ntessore

This would be nice, and is easy to do, but marking it "future" because it's also not essential for now.

ntessore avatar Jul 31 '23 15:07 ntessore

Hi, @ntessore! Do you have any strong preference for which coverage application to use? I was thinking Codecov because I really like its UI. IIRC, you'll need to add an API token (CODECOV_TOKEN) in the repository secrets on GitHub and enable a few settings (PR comments, annotations, badge) in Codecov to get everything working. I'll create a PR updating the workflow and config files.

Saransh-cpp avatar Sep 12 '24 13:09 Saransh-cpp

No preference for codecov or coveralls from my side, so I am happy to enable codecov right now for this repository.

ntessore avatar Sep 12 '24 13:09 ntessore

I have a mild preference for coveralls (I find codecov can be a bit noisy and false-positivey). However, we as a group, recommend both http://github-pages.arc.ucl.ac.uk/python-tooling/pages/ci.html#coverage-monitoring

paddyroddy avatar Sep 12 '24 13:09 paddyroddy

I'll go with coveralls, haven't used it before so would be a good learning opportunity

Saransh-cpp avatar Sep 12 '24 16:09 Saransh-cpp

Sent a coveralls request to organisation owners for approval ✅

Saransh-cpp avatar Sep 13 '24 10:09 Saransh-cpp