Skip to content

docs: make description of timeout_commit more precise#3542

Merged
cason merged 1 commit intomainfrom
cason/3089-timeout-commit
Jul 26, 2024
Merged

docs: make description of timeout_commit more precise#3542
cason merged 1 commit intomainfrom
cason/3089-timeout-commit

Conversation

@cason
Copy link

@cason cason commented Jul 22, 2024

Also, added reference to the new FinalizeBlock return value intended to replace the timeout_commit config flag.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments
  • Title follows the Conventional Commits spec

@cason cason added the documentation Improvements or additions to documentation label Jul 22, 2024
@cason cason requested review from a team as code owners July 22, 2024 14:36
@cason cason requested a review from a team July 22, 2024 14:36
@cason cason changed the title doc: make description of timeout_commit more precise docs: make description of timeout_commit more precise Jul 22, 2024
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@cason cason requested a review from melekes July 23, 2024 07:10
Copy link
Collaborator

@melekes melekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cason ❤️

@cason cason added this pull request to the merge queue Jul 26, 2024
Merged via the queue into main with commit 0ef00fb Jul 26, 2024
@cason cason deleted the cason/3089-timeout-commit branch July 26, 2024 12:27
melekes pushed a commit that referenced this pull request Dec 13, 2024
Also, added reference to the new `FinalizeBlock` return value intended
to replace the `timeout_commit` config flag.

---

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [x] Updated relevant documentation (`docs/` or `spec/`) and code
comments
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants