Skip to content

test rebase: Upgrade miniscript/bitcoin dependency#1405

Closed
notmandatory wants to merge 1 commit intobitcoindevkit:masterfrom
notmandatory:10-13-upgrade-bitcoin
Closed

test rebase: Upgrade miniscript/bitcoin dependency#1405
notmandatory wants to merge 1 commit intobitcoindevkit:masterfrom
notmandatory:10-13-upgrade-bitcoin

Conversation

@notmandatory
Copy link
Copy Markdown
Member

Upgrade:

  • bitcoin to v0.31.0
  • miniscript to v11.0.0

Note: The bitcoin upgrade includes improvements to the Transaction::weight() function, it appears those guys did good, we no longer need to add the 2 additional weight units "just in case".

Description

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

Upgrade:

- bitcoin to v0.31.0
- miniscript to v11.0.0

Note: The bitcoin upgrade includes improvements to the
`Transaction::weight()` function, it appears those guys did good, we
no longer need to add the 2 additional weight units "just in case".
@notmandatory notmandatory deleted the 10-13-upgrade-bitcoin branch April 12, 2024 16:07
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.

2 participants