-
Notifications
You must be signed in to change notification settings - Fork 142
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
Bump rust-bitcoin
to v0.32.5
#1487
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1487 +/- ##
==========================================
+ Coverage 18.28% 18.34% +0.05%
==========================================
Files 127 127
Lines 9738 9723 -15
==========================================
+ Hits 1781 1784 +3
+ Misses 7957 7939 -18
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Looks good, thats a big jump to 0.32.5. Added few comments.
I think by the end of the review it would be best to squash all commits into a single one as none of them compiles on its own.
looks like some recent PR merges introduced conflicts here |
f899c3f
to
d567a12
Compare
Solved. This PR is ready for review. |
d567a12
to
0c0f289
Compare
I was able to mine several signet blocks with the code from this PR http://75.119.150.111:8080/block/00000000003ef32fc7a8e7d0b376e626b7b1c825081ed82af1396d001f5b6fa1 here's the logs: pool
jds
jdc
tproxy
|
bef9d9c
to
0c4276e
Compare
This PR bumps
bitcoin
crate we use to its0.32.5
version.I grouped commits by categories of changes and adaptations:
uint256
removalBlock header
Hashes
Transaction
Script/ScriptBuf
I really recommend to review commit by commit, to make the work easier.
Close #1457