Skip to content

Merge master to develop#3563

Merged
ebuchman merged 5 commits intodevelopfrom
master
Apr 15, 2019
Merged

Merge master to develop#3563
ebuchman merged 5 commits intodevelopfrom
master

Conversation

@ebuchman
Copy link
Contributor

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

ebuchman and others added 5 commits April 2, 2019 19:18
 - prep changelog
 - add missing changelog entries
 - fix minor glitch in existing changelog (v0.31.2)
 - bump versions
- mention ADR in release summary
 - remove [p2p] api changes
 - amend v0.31.3 log to contain note about breaking change
@ebuchman ebuchman requested review from melekes and xla as code owners April 15, 2019 12:16
@ebuchman ebuchman merged commit d35c087 into develop Apr 15, 2019
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Sep 30, 2024
…(backport tendermint#3557) (tendermint#3563)

Closes tendermint#3544.

### Changes
- Adds missing `for` loop labels in the kv indexer 
- Previously ineffective `break` statements now point to their enclosing
`for` loop labels to exit upon reception on the `ctx.Done()` channel.

---

#### PR checklist

~- [ ] Tests written/updated~
- [x] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
~- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments~
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request tendermint#3557 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Alessandro <alessandro@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants