Skip to content
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 version to 0.17.0 #239

Merged
merged 2 commits into from Mar 1, 2019

Conversation

Projects
None yet
6 participants
@apoelstra
Copy link
Member

apoelstra commented Feb 28, 2019

No description provided.

@apoelstra apoelstra requested review from dongcarl , stevenroose and TheBlueMatt Feb 28, 2019

@dongcarl
Copy link
Member

dongcarl left a comment

Ship it!

@stevenroose
Copy link
Collaborator

stevenroose left a comment

Not sure if it's in this or previous release. But the PR adding derivation path parsing got merged recently. Might be worth mentioning.

@apoelstra

This comment has been minimized.

Copy link
Member Author

apoelstra commented Feb 28, 2019

It's in this release, the "BIP32 cleanup" link goes to it. Should I reword that?

Also @TheBlueMatt I'd like your ACK on this.

@stevenroose

This comment has been minimized.

Copy link
Collaborator

stevenroose commented Mar 1, 2019

@TheBlueMatt
Copy link
Member

TheBlueMatt left a comment

Sorry been kinda MIA of late given other projects, but I dont have any opposition to this in principle.

* [Clean up BIP32](https://github.com/rust-bitcoin/rust-bitcoin/pull/233) in preparation for PSBT; [use new native key types rather than `secp256k1` ones](https://github.com/rust-bitcoin/rust-bitcoin/pull/238/)
* Remove [apparently-used `Option` serialization](https://github.com/rust-bitcoin/rust-bitcoin/pull/236#event-2158116421) code
* Finally merge [PSBT](https://github.com/rust-bitcoin/rust-bitcoin/pull/103) after nearly nine months
# 0.16.0 - 2019-01-15

This comment has been minimized.

@dr-orlovsky

dr-orlovsky Mar 1, 2019

Contributor

Let me suggest adding to the list a recently merged PR for serialization for a network stream: #231

This comment has been minimized.

@apoelstra

apoelstra Mar 1, 2019

Author Member

Sure.

This comment has been minimized.

@apoelstra

apoelstra Mar 1, 2019

Author Member

Look good now?

@apoelstra apoelstra dismissed stale reviews from stevenroose and dongcarl via 8e214ff Mar 1, 2019

@apoelstra apoelstra merged commit 8aa964c into rust-bitcoin:master Mar 1, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@apoelstra apoelstra deleted the apoelstra:2019-02-0.17 branch Mar 1, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.