-
Notifications
You must be signed in to change notification settings - Fork 608
Upgrading solidity-coverage and using new plugin format #452
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
Conversation
@cgewecke do you happen to have any ideas on the above error with the linker? |
@justinjmoses Will take a look... |
@justinjmoses #453 has some possible fixes.... |
You're a rockstar @cgewecke 🚀. Do you have an address for donations? |
Codecov Report
@@ Coverage Diff @@
## develop #452 +/- ##
==========================================
Coverage ? 87.99%
==========================================
Files ? 47
Lines ? 1941
Branches ? 266
==========================================
Hits ? 1708
Misses ? 233
Partials ? 0 Continue to review full report at Codecov.
|
@justinjmoses Oh that's really sweet of you - solidity-coverage 0.7.x was funded by an EF grant and getting it to run on some of the larger, more widely used projects was part of the initial proposal. It's received all the donations it needs, thanks though :) |
Fixes coverage reports (thanks to @cgewecke) and adds codecov.io reports via travis