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
fix #1927: upgrade btcd #1990
fix #1927: upgrade btcd #1990
Conversation
Coverage from tests in coverage: 47.9% of statements across all listed packagescoverage: 52.9% of statements in consensus/istanbul coverage: 42.7% of statements in consensus/istanbul/announce coverage: 56.0% of statements in consensus/istanbul/backend coverage: 0.0% of statements in consensus/istanbul/backend/backendtest coverage: 24.3% of statements in consensus/istanbul/backend/internal/replica coverage: 56.4% of statements in consensus/istanbul/core coverage: 50.0% of statements in consensus/istanbul/db coverage: 0.0% of statements in consensus/istanbul/proxy coverage: 64.4% of statements in consensus/istanbul/uptime coverage: 51.8% of statements in consensus/istanbul/validator coverage: 79.2% of statements in consensus/istanbul/validator/randomCommentID: 69b72d95f7 |
Codecov ReportBase: 54.82% // Head: 54.81% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1990 +/- ##
==========================================
- Coverage 54.82% 54.81% -0.02%
==========================================
Files 688 695 +7
Lines 92542 93258 +716
==========================================
+ Hits 50737 51115 +378
- Misses 37993 38300 +307
- Partials 3812 3843 +31
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
LGTM!
I would add to the PR description that we also commented the ios compilation on circle (to have a better tracking)
Good point, created a tracking issue and mentioned it in the description. |
Description
Upgrading some dependencies - see #1927
Ported ethereum/go-ethereum#24533 and made some extra changes to make things work.
Also this caused problems with the ios build, which was therefor disabled. This is tracked in #2000.
Other changes
Some changes ported from ethereum/go-ethereum#24533 might not have been absolutely necessary (ex. fixing a typo).
Tested
Existing tests
Related issues
Backwards compatibility
According to #24533, the changes are perfectly compatible and affect only non-cgo builds anyway. Looks like celo-blockchain can't build without cgo yet, so this is definitely not going to be an issue.