-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add anvil tests to the coverage reports #246
Comments
@charles-cooper shall we use codecov? Could you add the token like on the vyper repo? |
i'm not really a huge fan of codecov, it often gives inaccurate reports. are there any alternatives? |
we can also do it manually, see https://stackoverflow.com/questions/26214055/combine-python-coverage-files |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We currently cover a lot of code via anvil integration tests, but those are not added to the code coverage reports.
I think it's nice to add the anvil coverage to the reports so that we know what parts of the code are not being tested at all.
The text was updated successfully, but these errors were encountered: