Updated CHANGES.md

[greg-szabo]

• Referenced an issue explaining the need for the change
• Updated all relevant documentation in docs
• Updated all code comments where relevant
• Wrote tests
• Updated CHANGES.md

Updates to CHANGES.md: added the changes implemented in the earlier Tendermint/v0.33 change.

[codecov-io]

Codecov Report

Merging #243 into master will increase coverage by 0.0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #243   +/-   ##
======================================
  Coverage    39.4%   39.5%           
======================================
  Files          99      99           
  Lines        3641    3633    -8     
  Branches      587     587           
======================================
- Hits         1438    1437    -1     
+ Misses       1798    1791    -7     
  Partials      405     405           

Impacted Files	Coverage Δ
tendermint/src/lite_impl/signed_header.rs	68.7% <0.0%> (-0.4%)	⬇️
tendermint/src/rpc/id.rs	0.0% <0.0%> (ø)
tendermint/src/version.rs	0.0% <0.0%> (ø)
tendermint/src/amino_types/proposal.rs	33.0% <0.0%> (+0.2%)	⬆️
tendermint/src/public_key.rs	52.1% <0.0%> (+0.4%)	⬆️
tendermint/src/net.rs	37.7% <0.0%> (+1.7%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26d81be...25a31dc. Read the comment docs.

[liamsi]

Thanks Greg!

[liamsi]

Note that integration tests are failing again (on master and all branches), since tendermint 0.33.4 was released (see #233). #238 fixes this.

[liamsi]

Question: don't we want to keep referencing and linking to issues consistent? See previous entries:
https://github.com/informalsystems/tendermint-rs/pull/243/files#diff-8b1c3fd0d4a6765c16dfd18509182f9dR18
https://github.com/informalsystems/tendermint-rs/pull/243/files#diff-8b1c3fd0d4a6765c16dfd18509182f9dR21

[liamsi]

Same as above (referencing and linking).

[liamsi]

Not sure how relevant CI changes are to readers of the changelog because these do not have any effect on users of tendermint-rs. On the other hand it's probably good to communicate what happened since the last release anyways. Note that the other badges were also updated in a another PR/issue.

[greg-szabo]

Updated the CHANGES file to match the format previously agreed. Please review.

[liamsi]

👍

[liamsi]

Note that links are done slightly different in this PR than previously, e.g see:

## [0.13.0] (2020-04-20)

Dependencies:
- Update `signatory` requirement to v0.19 ([#227])

[0.13.0]: https://github.com/informalsystems/tendermint-rs/pull/228
[#227]: https://github.com/informalsystems/tendermint-rs/pull/227

Now the links are directly embedded in the text (rendered both look the same; the former is easier to read if not rendered though).
Something to keep in mind, if we ever automate adding to the changelog.