rust-bitcoin / rust-bitcoin / 7612215536

83%

push

github

Merge rust-bitcoin/rust-bitcoin#2268: Add convenience constants to `Denomination`

<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/22747149a9040c075e3f1f4eec8500f31b515659">22747149a Add convenience constants to `Denomination` (Martin Habovstiak)

Pull request description:

  `Denomination::Bitcoin` and `Denomination::Satoshi` are often used, especially in test code so this change adds `BTC` and `SAT` - short, readable constants. Notably this doesn't add the other constants as that would lead to either unidiomatic names or confusing casing (MSAT meaning millisat not megasat) and they are not used that much anyway.

ACKs for top commit:
  apoelstra:
    ACK 22747149a9040c075e3f1f4eec8500f31b515659
  tcharding:
    ACK 22747149a9040c075e3f1f4eec8500f31b515659

Tree-SHA512: fb49e61c2e96798d3556b4d4f6eb9c87da1

19229 of 22820 relevant lines covered (84.26%)

16915.19 hits per line