rust-bitcoin / rust-bitcoin / 14131948505
83%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 129
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 83.543% (+0.05%) from 83.491%
14131948505

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#4281: Return `ControlBlock` from `Witness::taproot_control_block`

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/492073f28876406f8fe5a07a8a2495c8e0ba1fb3">492073f28<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/a2408e9b0cec754525280f83b349d8d933d92927"> Strengthen the type of `taproot_control_block()` (Martin Habovstiak)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/e8a42d5851000d210ea9b886478d83f3e7c077a6">e8a42d585</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/a2408e9b0cec754525280f83b349d8d933d92927"> Unify/reduce usage of `unsafe` (Martin Habovstiak)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/d42364bd9d39ce4d509cf2d13336bc3bc619e3f1">d42364bd9</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/a2408e9b0cec754525280f83b349d8d933d92927"> Swap around the fields in `Address` (Martin Habovstiak)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/7a115e3cf13f5cde24b957efd888b0c0c6840e2c">7a115e3cf</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/a2408e9b0cec754525280f83b349d8d933d92927"> Make `Address` obey sanity rules (Martin Habovstiak)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/bc6da1fe07937051ada4a60c78f5f5128484a91a">bc6da1fe0 Swap around the fields in `sha256t::Hash` (Martin Habovstiak)
8ee088df7 Make `sha256t` obey sanity rules (Martin Habovstiak)

Pull request description:

  Well, I thought this PR will be just the last commit... :sweat_smile:

  Anyway, this implements a bunch of changes to allow returning `ControlBlock` from `Witness` method(s). One cool side effect is that this PR also reduces the number of `unsafe` blocks.

ACKs for top commit:
  apoelstra:
    ACK 492073f28876406f8fe5a07a8a2495c8e0ba1fb3; successfully ran local tests

Tree-SHA512: 11979517c8ba2d9b402ca850c112979637e

92 of 131 new or added lines in 10 files covered. (70.23%)

9 existing lines in 2 files now uncovered.

22230 of 26609 relevant lines covered (83.54%)

16847.77 hits per line

Uncovered Changes

Lines Coverage ∆ File
15
67.17
 -1.04% bitcoin/src/crypto/key.rs
14
51.22
 51.22% internals/src/macros.rs
6
92.24
 0.09% bitcoin/src/address/mod.rs
2
77.85
 2.32% primitives/src/script/mod.rs
1
1.08
 0.0% bitcoin/src/address/error.rs
1
44.19
 3.76% hashes/src/sha256t/mod.rs

Coverage Regressions

Lines Coverage ∆ File
6
85.9
 -0.37% bitcoin/src/psbt/mod.rs
3
41.67
 -5.0% bitcoin/src/taproot/merkle_branch/buf.rs
Jobs
ID Job ID Ran Files Coverage
1 14131948505.1 129
83.54
 GitHub Action Run
Source Files on build 14131948505