bitcoindevkit / bdk / 10780014759

79%

push

github

Merge bitcoindevkit/bdk#1595: refactor(wallet): use `Amount` everywhere

<a class=hub.com/bitcoindevkit/bdk/commit/<a class="double-link" href="https://git"><a class=hub.com/bitcoindevkit/bdk/commit/292ec3cb3a41d7b9f42cfea536c39b27dba16d4c">292ec3cb3 refactor(wallet): use `Amount` everywhere (valued mammal)

Pull request description:

  This is a followup to #1426 that refactors wallet internals to use `bitcoin::Amount` (nearly) everywhere. I chose not to change any public types in `coin_selection.rs` that still use `u64` as that might require more discussion.

  partially addresses #1432
  fixes #1434

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

ACKs for top commit:
  oleonardolima:
    ACK 292ec3cb3a41d7b9f42cfea536c39b27dba16d4c
  notmandatory:
    ACK 292ec3cb3a41d7b9f42cfea536c39b27dba16d4c

Tree-SHA512: e84c543e74430dd6c6152c3146bca7e3be4

20 of 20 new or added lines in 3 files covered. (100.0%)

11099 of 13580 relevant lines covered (81.73%)

14506.67 hits per line