-
Notifications
You must be signed in to change notification settings - Fork 831
bump version to 0.25.1 #491
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
need to add #480 to changelog |
abcd0ac
to
17e733d
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.
NACK: 3618d7a was a breaking change, so it can't be 0.25.1 anymore. See rust-bitcoin/rust-miniscript#153
Hmm that method returns a |
Here: #493 |
Ok #493 is merged |
Is it possible to release this version on Monday pls? Any more reviewers, @stevenroose, @elichai? |
@dr-orlovsky I would like to get #413 in first. |
Should maybe add #500 to changelog |
Should add #413 to changelog |
Co-authored-by: Dr. Maxim Orlovsky <orlovsky@pandoracore.com>
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.
I didnt review anything in the bump, but, like, the version number change looks like a version number change.
…e additional methods for converting to a DescriptorPublicKey 1477dbb DefiniteDescriptorKey: provide additional methods for converting to a DescriptorPublicKey (Andrew Poelstra) Pull request description: Fixes rust-bitcoin#491 I didn't provide a mutable version of this. I'm not sure if we want to support this or not; generally public keys should be immutable after creation. ACKs for top commit: sanket1729: ACK 1477dbb Tree-SHA512: 8c06ac3b6c395c3805b8cbd958ce4f354025ee88b2915562662cb084216048c6543591c468112981e8afa72d89373a544965096227475f0c198af9194d826a7e
No description provided.