-
Couldn't load subscription status.
- Fork 70
feat: BIP-0094 "testnet4" #494
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
|
Updated with @JoseSK999's comments |
|
Was this able to sync with the bridge node? |
|
I guess there is no bridge yet, I'll try to get one running over the weekend. |
It headers-synced. But I'll try with a proper bridge node here and report back. |
|
Oh actually headers sync is now testing the full change behavior |
|
I've just remembered, |
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.
Ok so some final comments
This commit implements testnet4-related pow changes, and makes sure that all node functionalities works with it.
|
rebased after #506 and updated with @JoseSK999's comments. |
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.
ACK 8546ba3
|
@Davidson-Souza does your bridge work with testnet4? |
|
Answer: not really. |
What is the purpose of this pull request?
Which crates are being modified?
Description
Closes #156
This commit implements BIP-0094-related pow changes, and makes sure that all node functionalities works with it the "testnet4" network.
Note to reviwers
I've managed to headers-sync, but didn't download blocks due to no bridges so far. I'll try to get one running