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

Add to and from base64 string to pset #175

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

RCasatta
Copy link
Collaborator

via standard trait Display and FromStr like it's done in rust-bitcoin PSBT.

Reuse rust-bitcoin base64 re-exported

@RCasatta RCasatta changed the title Add to and from base64 string o pset Add to and from base64 string to pset Sep 14, 2023
@RCasatta RCasatta force-pushed the pset_str branch 2 times, most recently from 86f2ae8 to 5f8fb1b Compare September 14, 2023 08:55
@sanket1729
Copy link
Member

@RCasatta, there is a clippy warning. For the MSRV fixes, you can refer to the rust-bitcoin cargo.toml file.

via standard trait Display and FromStr like it's done in rust-bitcoin PSBT.

Reuse rust-bitcoin base64 re-exported
@RCasatta
Copy link
Collaborator Author

@sanket1729 CI for 1.48.0 and clippy fixed

Copy link
Member

@delta1 delta1 left a comment

Choose a reason for hiding this comment

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

ACK f42203e

Copy link
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

ACK f42203e

@sanket1729 sanket1729 merged commit e9a4145 into ElementsProject:master Sep 20, 2023
5 checks passed
sanket1729 added a commit that referenced this pull request Sep 22, 2023
ed066af rename pset::str::Error to ParseError and expose it (Riccardo Casatta)

Pull request description:

  Otherwise the pset decoding from string cannot be used downstream

  fix #175

ACKs for top commit:
  apoelstra:
    ACK ed066af
  sanket1729:
    ACK ed066af

Tree-SHA512: 7ee8346ddea5b6bfa21bdc8fbf0a4fb23ab056821625f0bb67b7b8c271b3c6570f2c431fc0f23f6ee04c025f240d0b7a72a3849e64cd143de4563bfba9634721
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants