Ran
|
Jobs
1
|
Files
39
|
Run time
4s
|
Badge
Embed ▾
README BADGES
|
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
ID | Job ID | Ran | Files | Coverage | |
---|---|---|---|---|---|
1 | 7728057127.1 | 39 |
81.1 |
GitHub Action Run |
Coverage | ∆ | File | Lines | Relevant | Covered | Missed | Hits/Line |
---|