Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Remove tendermint engine support #9980

Merged
merged 5 commits into from
Nov 28, 2018
Merged

Remove tendermint engine support #9980

merged 5 commits into from
Nov 28, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Nov 28, 2018

Our current tendermint engine is unmaintained, and was reported to be broken. This PR removes tendermint engine code. Integration with rhododendro BFT as replacement for tendermint is planned in the (really short-term) future.

@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 28, 2018
@sorpaas sorpaas added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 28, 2018
@5chdn 5chdn added this to the 2.3 milestone Nov 28, 2018
@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Nov 28, 2018
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 28, 2018
@5chdn 5chdn added the B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release. label Nov 28, 2018
@sorpaas sorpaas merged commit f092c10 into master Nov 28, 2018
@sorpaas sorpaas deleted the sp-remove-tendermint branch November 28, 2018 22:47
niklasad1 pushed a commit that referenced this pull request Dec 16, 2018
* Remove tendermint engine support

* Remove tendermint test json spec

* Fix ethcore test compile

* Remove tendermint test in sync
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants