bitcoindevkit / bdk_wallet / 23327736848

86%

master: 85%

push

github

Merge bitcoindevkit/bdk_wallet#399: Backport: Add apply_unconfirmed_txs_events and apply_evicted_txs_events

<a class=hub.com/bitcoindevkit/bdk_wallet/commit/ff7c15d2b931b4977660644f4234efcd41ed940b">ff7c15d2b<a href="https://github.com/bitcoindevkit/bdk_wallet/commit/7f22aa7e6f5afe3bc257b3a1d7997f063c4132cd"> ci: pin dependencies for msrv 1.63.0 (Steve Myers)
<a class="double-link" href="https://github.com/bitcoindevkit/bdk_wallet/commit/c4fc5d83030d42805360cadc191db47adf352a3c">c4fc5d830</a><a href="https://github.com/bitcoindevkit/bdk_wallet/commit/7f22aa7e6f5afe3bc257b3a1d7997f063c4132cd"> docs(wallet): fix events_helper E fmt, docs on event::wallet_events (Steve Myers)
</a><a class="double-link" href="https://github.com/bitcoindevkit/bdk_wallet/commit/3cc9a919fc9eb633ae500c9800b15c7bb2bf4dff">3cc9a919f</a><a href="https://github.com/bitcoindevkit/bdk_wallet/commit/7f22aa7e6f5afe3bc257b3a1d7997f063c4132cd"> feat(wallet): Add `Wallet::events_helper` method (valued mammal)
</a><a class="double-link" href="https://github.com/bitcoindevkit/bdk_wallet/commit/ed0bb912d397191e116ca07299b36ef4d5f59718">ed0bb912d</a><a href="https://github.com/bitcoindevkit/bdk_wallet/commit/7f22aa7e6f5afe3bc257b3a1d7997f063c4132cd"> feat(wallet): add apply_unconfirmed_txs_events and apply_evicted_txs_events (Steve Myers)

Pull request description:

  ### Description

  Backport #388 to `release/2.x` branch.

  ### Notes to the reviewers

  I had to make some small changes in </a><a class="double-link" href="https://github.com/bitcoindevkit/bdk_wallet/commit/57401507294e346ea758db8a55fa591ea80473b7">574015072</a> to fix differences in 2.x use statements related to `fmt` and that `WalletEvents` isn't exposed at the module level.

  ### Changelog notice

  - Add Wallet::apply_unconfirmed_txs_events
  - Add Wallet::apply_evicted_txs_events
  - Add Wallet::events_helper function for advanced users

  ### 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 `just p` before pushing

  #### New Features:

  * [x] I've added tests for the new feature
  * [x] I've added docs for the new feature

ACKs for top commit:
  ValuedMammal:
    ACK ff7c15d2b931b4977660644f4234efcd41ed940b; ran the tests locally

Tree-SHA512: 7029295db47a118ee6a5ae40af704aea96e

57 of 58 new or added lines in 2 files covered. (98.28%)

7142 of 8299 relevant lines covered (86.06%)

3145.36 hits per line