Skip to content

revert(config)!: merge timeout_prevote and timeout_precommit#3177

Merged
andynog merged 1 commit intov1.xfrom
3014-revert
Jun 4, 2024
Merged

revert(config)!: merge timeout_prevote and timeout_precommit#3177
andynog merged 1 commit intov1.xfrom
3014-revert

Conversation

@melekes
Copy link
Collaborator

@melekes melekes commented Jun 4, 2024

…` (backport #2895) (#3014)"

This reverts commit 54a086b.


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

@melekes melekes requested a review from a team as a code owner June 4, 2024 08:12
@melekes melekes requested a review from a team June 4, 2024 08:12
@melekes melekes changed the title Revert "feat(config)!: merge timeout_prevote and `timeout_precommit… revert(config)!: merge timeout_prevote and timeout_precommit Jun 4, 2024
Copy link
Contributor

@adizere adizere left a comment

Choose a reason for hiding this comment

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

Thank you Anton!

Once @andynog also finishes a pass through the diff LGTM!

Copy link
Collaborator

@andynog andynog left a comment

Choose a reason for hiding this comment

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

the revert looks good, we checked the diffs and all the changes between the original merge and the revert are kept properly

@andynog andynog merged commit e1d6a33 into v1.x Jun 4, 2024
@andynog andynog deleted the 3014-revert branch June 4, 2024 09:08
@melekes melekes added the breaking A breaking change label Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking A breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants