bitcoindevkit / bdk / 7728057127

79%

push

github

Merge bitcoindevkit/bdk#1306: Implement `ElectrumExt` for all that implements `ElectrumApi`

<a class=hub.com/bitcoindevkit/bdk/commit/ce8c617c9d562a54728c6cafc0562a66bae56b3e">ce8c617c9<a href="https://github.com/bitcoindevkit/bdk/commit/f099b42005b5310cee51fa2eaa29f94d76753bd7"> feat(electrum): impl `ElectrumExt` for all that impl `ElectrumApi` (志宇)

Pull request description:

  ### Description

  Implement `ElectrumExt` for all that implements `ElectrumApi`.

  I realized this while looking into #1171.

  https://github.com/bitcoindevkit/bdk/blob/<a class="double-link" href="https://github.com/bitcoindevkit/bdk/commit/2196685cca741a249f4a5b53c7609162469714ba">2196685cc</a>/crates/electrum/tests/test_electrum.rs#L54-L55

  Line 55 here should not be necessary.

  ### Changelog notice

  * Changed to implement `ElectrumExt` for all that implements `ElectrumApi`.

  ### 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:
  notmandatory:
    ACK ce8c617c9d562a54728c6cafc0562a66bae56b3e

Tree-SHA512: d95193a9a520f52547ffdb0bf95f26ab937

9225 of 11375 relevant lines covered (81.1%)

5700.55 hits per line