-
Notifications
You must be signed in to change notification settings - Fork 790
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
MPT: New Release v4.1.0 #1108
MPT: New Release v4.1.0 #1108
Conversation
…favor of deprecation
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
0c3e67a
to
fe1512c
Compare
fe1512c
to
0aee672
Compare
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 💯
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!
Already reviewed and just updated the branch on tiny changes, will admin-merge for practical reasons. |
Just released this as v4.1.0 on npm. |
MPT release with the new checkpointing mechanism and other CI and docs related changes (see CHANGELOG).
Please notice that I reverted a
Trie.setRoot()
->Trie._setRoot()
rename (introduced by myself) from ethereumjs/merkle-patricia-tree#125 and switched to deprecating instead. This seemed to dangerous to me on retrospective regarding introducing a breaking change. Have instead added to #1024.