rust-bitcoin / rust-bitcoin / 9959794868

83%

push

github

Merge rust-bitcoin/rust-bitcoin#3019: Fix error messages

<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/2169b75bba5cde8a23681ca9971cc40c343b9e17">2169b75bb Use lower case error messages (Jamil Lambert, PhD)

Pull request description:

  Error messages should be lower case, except for proper nouns and variable names.  These have all been changed.

  ~~They should also state what went wrong.  Some expect error messages were positive, giving the correct behaviour or correct input.  These have been changed so that they are now negative, i.e. saying what went wrong.~~

  EDIT: After further discussion it was decided not to change the expect messages.

ACKs for top commit:
  Kixunil:
    ACK 2169b75bba5cde8a23681ca9971cc40c343b9e17
  tcharding:
    ACK 2169b75bba5cde8a23681ca9971cc40c343b9e17

Tree-SHA512: 92442c869f68c4230ac30728052d007021c

19 of 36 new or added lines in 16 files covered. (52.78%)

19645 of 23611 relevant lines covered (83.2%)

15654.25 hits per line