rust-bitcoin / rust-bitcoin / 16325990762
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.5% (-0.02%) from 83.524%
16325990762

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#4710: Update NumOpResult arbitrary impl to only consume 1 byte of entropy

<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/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/453888a33c4f496f2309307a41a06667a4bc0637">453888a33 Update NumOpResult arbitrary impl to only consume 1 byte of entropy (Shing Him Ng)

Pull request description:

  Took some inspiration from [a comment](https://github.com/rust-bitcoin/rust-bitcoin/pull/4689#issuecomment-3073834482) in #4689 and looked through the rest of the codebase to update `u.int_in_range(0..=1)` to use `bool::arbitrary(u)`


ACKs for top commit:
  yancyribbens:
    cr ACK 453888a33c4f496f2309307a41a06667a4bc0637
  tcharding:
    ACK 453888a33c4f496f2309307a41a06667a4bc0637
  apoelstra:
    ACK 453888a33c4f496f2309307a41a06667a4bc0637; successfully ran local tests


Tree-SHA512: 2ee44d4bd9fb186b545ed1b030ead62381c

0 of 3 new or added lines in 1 file covered. (0.0%)

7 existing lines in 1 file now uncovered.

21275 of 25479 relevant lines covered (83.5%)

15265.17 hits per line

Uncovered Changes

Lines Coverage ∆ File
3
64.41
 0.54% units/src/result.rs

Coverage Regressions

Lines Coverage ∆ File
7
86.65
 -0.46% bitcoin/src/psbt/mod.rs
Jobs
ID Job ID Ran Files Coverage
1 16325990762.1 129
83.5
 GitHub Action Run
Source Files on build 16325990762