Skip to content

Change PublicKey::write_into to return Result#507

Merged
apoelstra merged 7 commits intorust-bitcoin:masterfrom
stevenroose:pubkey-write-result
Nov 5, 2020
Merged

Change PublicKey::write_into to return Result#507
apoelstra merged 7 commits intorust-bitcoin:masterfrom
stevenroose:pubkey-write-result

Conversation

@stevenroose
Copy link
Copy Markdown
Collaborator

Followup on #506.

@stevenroose stevenroose added the API break This PR requires a version bump for the next release label Oct 23, 2020
@stevenroose stevenroose changed the title Pubkey write result Change PublicKey::write_into to return Result Oct 23, 2020
Copy link
Copy Markdown
Member

@elichai elichai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@apoelstra apoelstra merged commit d3210b3 into rust-bitcoin:master Nov 5, 2020
yancyribbens pushed a commit to yancyribbens/rust-bitcoin that referenced this pull request Mar 23, 2024
yancyribbens pushed a commit to yancyribbens/rust-bitcoin that referenced this pull request Mar 23, 2024
…coin#507

f0f7fd6 fix formatting after rust-bitcoin#507 (Andrew Poelstra)

Pull request description:

  I merged rust-bitcoin#507 without checking CI, and it turns out there was a rustfmt issue. FIx that.

ACKs for top commit:
  tcharding:
    ACK f0f7fd6
  sanket1729:
    ACK f0f7fd6

Tree-SHA512: 909d0a118c0e36f40b7aa6f0d6c7718b1b77b00acbb07ae0183a26e107a24f477debebc701545a8295a31dc589b9f40174a44a0c6e935ecab445c447078c3784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API break This PR requires a version bump for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants