bitcoindevkit
/
bdk
/
10033703097
/
1
79%
master: 79%
master: 79%
|
Ran
|
Files
46
|
Run time
1s
|
Badge
Embed ▾
README BADGES
|
coverage: 81.881% (+0.005%) from 81.876%
push
github
notmandatoryMerge bitcoindevkit/bdk#1506: Standardize API ownership in `KeychainTxOutIndex` 79262185d refactor(chain)!: update KeychainTxOutIndex methods to use owned ScriptBuf (Steve Myers) 7c07b9de0 refactor(chain)!: update KeychainTxOutIndex methods to use owned K (Rob N) Pull request description: ### Description Make all method signatures of `KeychainTxOutIndex` take owned `K` and use `ScriptBuf` instead of its borrowed counterpart `&Script`. Fixes #1482 ### Notes to the reviewers Steve also added a CI fix as well ### Changelog notice - Make all method signatures of `KeychainTxOutIndex` take owned `K` - Update `KeychainTxOutIndex` methods to use `ScriptBuf` ### 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 #### New Features: * [ ] I've added tests for the new feature * [ ] I've added docs for the new feature #### Bugfixes: * [x] This pull request breaks the existing API * [ ] I've added tests to reproduce the issue which are now passing * [x] I'm linking the issue being fixed by this PR Top commit has no ACKs. Tree-SHA512: 3cb7d627ee3d3d56cc1dac1f4677cf6b7b7
10950 of 13373 relevant lines covered (81.88%)
16514.44 hits per line
