rust-bitcoin / rust-bitcoin / 14892238074
83%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 131
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: 84.257% (+0.03%) from 84.223%
14892238074

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#4428: Use NumOpResult instead of Option

<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/13cbead94766987f59482b1fbc1d0ebd0799737c">13cbead94<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/884dc5e103bba51dd6980cabc404c2be0bd816dd"> Use NumOpResult instead of Option (yancy)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/002a0382aa8ce6bcf155ab29a6a2d8438e9ad108">002a0382a</a> Mark function constant (yancy)

Pull request description:

  Prefer the more descriptive NumOpResult return type over Option where return types are fallible.

  Closes https://github.com/rust-bitcoin/rust-bitcoin/issues/4419

ACKs for top commit:
  apoelstra:
    ACK 13cbead94766987f59482b1fbc1d0ebd0799737c; successfully ran local tests
  tcharding:
    ACK 13cbead94766987f59482b1fbc1d0ebd0799737c
  Kixunil:
    ACK 13cbead94766987f59482b1fbc1d0ebd0799737c

Tree-SHA512: 1a870962d1a4b3efef9034ab001df907c96

12 of 18 new or added lines in 3 files covered. (66.67%)

1 existing line in 1 file now uncovered.

23024 of 27326 relevant lines covered (84.26%)

16193.0 hits per line

Uncovered Changes

Lines Coverage ∆ File
6
87.3
 -1.41% units/src/fee.rs

Coverage Regressions

Lines Coverage ∆ File
1
86.38
 0.31% bitcoin/src/psbt/mod.rs
Jobs
ID Job ID Ran Files Coverage
1 14892238074.1 131
84.26
 GitHub Action Run
Source Files on build 14892238074