Skip to content

Conversation

@andozw
Copy link
Contributor

@andozw andozw commented Nov 23, 2024

Adding tests for the multisig quorum parameter conditions.

@andozw andozw force-pushed the seana.20241121.th1 branch 2 times, most recently from d07acf0 to 8f1c11d Compare November 23, 2024 00:46
@ivmaykov
Copy link
Contributor

ivmaykov commented Dec 2, 2024

@andozw did you mean to merge into a non-master branch?

Base automatically changed from seana.20241121.th1 to master December 2, 2024 19:17
@andozw
Copy link
Contributor Author

andozw commented Dec 2, 2024

@andozw did you mean to merge into a non-master branch?

Compare to non-master branch, yes. I created this branch off of the PR that I just merged. And github will automatically update the base branch when it gets merged.

ivmaykov
ivmaykov previously approved these changes Dec 2, 2024
Copy link
Contributor

@ivmaykov ivmaykov left a comment

Choose a reason for hiding this comment

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

LGTM with some questions/comments

@andozw andozw force-pushed the seana.20241121.testDeriveP2SHP2WSH branch 2 times, most recently from 1e344da to 006e746 Compare December 2, 2024 22:06
Adding tests for the multisig quorum parameter conditions.
@andozw andozw force-pushed the seana.20241121.testDeriveP2SHP2WSH branch from 006e746 to 92b4797 Compare December 2, 2024 22:19
Copy link
Contributor

@ivmaykov ivmaykov left a comment

Choose a reason for hiding this comment

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

Merge it!

@andozw andozw merged commit 4f270a0 into master Dec 2, 2024
11 checks passed
@andozw andozw deleted the seana.20241121.testDeriveP2SHP2WSH branch December 2, 2024 23:16
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