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
Update solidity-coverage to v0.7 #255
Conversation
I don't see a CI job for this, do you know why? |
Co-Authored-By: Brett Sun <qisheng.brett.sun@gmail.com>
For some reason the |
Do you mean in the CI? Because in locally it was working for me. I don't see gas tests in the coverage task either. Where are you seeing them? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's see how we can handle the travis timeout, not sure if there is a way to instrument the code in batches
Being coverage I don't know how we can batch it. What I would do for now is to remove the travis job and merge this PR, so at least we can run coverage locally. |
We could do a quick experiment or two using lcov-result-merger (see aragon-apps for usage). I think it even supports merging coverage results across different runs if the runs overlap on the same files. |
Closing this for now |
Fixes #86