Skip to content

feat: add max_weight_to_satisfy to Descriptor#794

Merged
reez merged 1 commit intobitcoindevkit:masterfrom
reez:max-satisfy
Jul 7, 2025
Merged

feat: add max_weight_to_satisfy to Descriptor#794
reez merged 1 commit intobitcoindevkit:masterfrom
reez:max-satisfy

Conversation

@reez
Copy link
Copy Markdown
Collaborator

@reez reez commented Jun 30, 2025

Description

https://docs.rs/bdk_wallet/latest/bdk_wallet/descriptor/enum.Descriptor.html#method.max_weight_to_satisfy

Related to #329 but has standalone value.

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

@reez reez marked this pull request as ready for review July 1, 2025 13:59
@reez reez changed the title (draft) feat: add max_weight_to_satisfy to Descriptor feat: add max_weight_to_satisfy to Descriptor Jul 1, 2025
@reez reez requested a review from ItoroD July 1, 2025 13:59
Copy link
Copy Markdown
Collaborator

@ItoroD ItoroD left a comment

Choose a reason for hiding this comment

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

ACK c6c8ae9

Great stuff! This will be a plus in helping estimate fees.

@reez reez merged commit 65cd4cd into bitcoindevkit:master Jul 7, 2025
24 checks passed
@reez reez deleted the max-satisfy branch July 7, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants