rust-bitcoin / rust-bitcoin / 15621162840
83%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 130
Run time 160min
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.352% (-0.03%) from 84.385%
15621162840

push

github

apoelstra 
Merge rust-bitcoin/rust-bitcoin#4614: Add fee rate constructors that take `Amount` as arg

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/6ed3fd6234a6d9e44d2cf187108cc22955471d27">6ed3fd623<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/aab9c2dfca6593409521abcb5415b86af65a7fcd"> Add fee rate constructors that take Amount as arg (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/c1a760bf603e49931425a1b6a6ba2b8450c8da11">c1a760bf6</a> units: Use singular in rustdoc (Tobin C. Harding)

Pull request description:

  Some users may find it more ergonomic to pass in an `Amount` when constructing fee rates. Also the strong type adds some semantic meaning as well as imposes the `Amount::MAX` limit.

  Add an equivalent constructor for each of the existing ones that uses an argument of type `Amount` instead of `u64` sats.

  (This was pulled out of #4610.)

  Close: #4734

ACKs for top commit:
  apoelstra:
    ACK 6ed3fd6234a6d9e44d2cf187108cc22955471d27; successfully ran local tests

Tree-SHA512: 92609457b2ee984e4ea2fee57c679c2afb1

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

23541 of 27908 relevant lines covered (84.35%)

15851.76 hits per line

Uncovered Changes

Lines Coverage ∆ File
18
80.1
 -7.67% units/src/fee_rate/mod.rs
Jobs
ID Job ID Ran Files Coverage
1 15621162840.1 130
84.35
 GitHub Action Run
Source Files on build 15621162840