-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code coverage in CI #52
Comments
So, after a little bit of trying out According to the README, one can merge different coverages based on features. The issue with this is that the different crypto backends are three different workflows (three links, for the three workflows, I'm not being passive aggressive or anything). On a recent code sample I used codecov.io/, I was pleased how nicely it consumed the reports from Anyway, if it's reasonable, I'd suggest refactoring the three crypto backend CI's into one CI that:
|
I randomly found myself looking at this issue last week and decided to try out some new tools for my CI for work without relying on external websites like codecov.io (I think it was recently acquired). Some actions I found useful are:
|
It would be useful for us to have code coverage in CI, especially for the crypto backends.
Depending on what service we use, this might require some integrating the repo to an external service, e.g. coveralls.
I can work on the Github Actions part and figure out how coverage will work for mutually exclusive targets, then I can leave some instructions for how a repo admin can enable CI on a service.
The text was updated successfully, but these errors were encountered: