rust-bitcoin / rust-bitcoin / 8397717447
83%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 91
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.444% (+0.02%) from 83.425%
8397717447

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#2619: Call is_witness_program from witness_version

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/dec05b63e9337fe93ab22e68ce544cc5d7db5dcf">dec05b63e<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/1656e1a09a1959230e20af90d20789a4a8f0a31b"> Refactor witness_version and is_witness_program (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/dac552b4365e9feaffc5614979c7c57d1f41ff5b">dac552b43</a> Add unit tests for shortest/longest witness program (Tobin C. Harding)

Pull request description:

  Refactor `witness_version` and `is_witness_program`.

  - Patch 2 adds a couple of preparatory unit tests.
  - Patch 2 does the refactor

  Fix: #2618

ACKs for top commit:
  apoelstra:
    ACK dec05b63e9337fe93ab22e68ce544cc5d7db5dcf
  sanket1729:
    ACK dec05b63e9337fe93ab22e68ce544cc5d7db5dcf

Tree-SHA512: 3db0a1d81620b12f082196a36d1793ca370

35 of 36 new or added lines in 1 file covered. (97.22%)

1 existing line in 1 file now uncovered.

19061 of 22843 relevant lines covered (83.44%)

16980.57 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
78.69
 1.49% bitcoin/src/blockdata/script/borrowed.rs

Uncovered Existing Lines

Lines Coverage ∆ File
1
32.18
 1.15% bitcoin/src/blockdata/script/witness_version.rs
Jobs
ID Job ID Ran Files Coverage
1 8397717447.1 91
83.44
 GitHub Action Run
Source Files on build 8397717447