Skip to content

Fix inconsistent naming in the dummy key hash structs #463

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

Merged

Conversation

afilini
Copy link
Contributor

@afilini afilini commented Sep 26, 2022

Also derive Default for DummyHash and all of its hash types since they are all empty structs.

@apoelstra
Copy link
Member

CI failure is a broken dependency, fixed in a commit of #461

sanket1729 and others added 2 commits September 26, 2022 15:39
Also derive `Default` for `DummyHash` and all of its hash types since
they are all empty structs.
@afilini afilini force-pushed the fix/inconsistent-dummy-hash-names branch from eb2c282 to af5d2d2 Compare September 26, 2022 13:39
@afilini
Copy link
Contributor Author

afilini commented Sep 26, 2022

I cherry-picked Sanket's commit and rebased on top of it

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK eb2c282

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK af5d2d2

@sanket1729
Copy link
Member

FTR, we plan to remove the DummyKey* types in the future. You should be able to test everything with MiniscriptKey = String type.

@sanket1729 sanket1729 merged commit b6daa97 into rust-bitcoin:master Sep 26, 2022
@afilini afilini deleted the fix/inconsistent-dummy-hash-names branch September 28, 2022 10:46
sanket1729 added a commit to sanket1729/elements-miniscript that referenced this pull request Oct 21, 2022
…022_10

b6daa97 Merge rust-bitcoin/rust-miniscript#463: Fix inconsistent naming in the dummy key hash structs

Export structs from upstream instead
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.

3 participants