Skip to content
This repository has been archived by the owner on Aug 13, 2019. It is now read-only.

Pluggable Consensus #45

Open
soc1c opened this issue Jun 4, 2019 · 3 comments
Open

Pluggable Consensus #45

soc1c opened this issue Jun 4, 2019 · 3 comments
Labels
ISS: Discussion Priority: 7 Low Type: Feature

Comments

@soc1c
Copy link
Contributor

soc1c commented Jun 4, 2019

For some reason ethereumproject/go-ethereum#648 was not merged in master but development branch 🤦‍♀️

We should try cherry-picking / rebasing on that PR

Required for #19 / Kotti
@austinabell
Copy link
Contributor

That PR just has a bunch of code copied from ETH with all of the code added referencing ethereum/go-ethereum. It is not functional and not tested.

@austinabell
Copy link
Contributor

The changes were barely WIP and I'm not sure why they were merged in and to add these changes to our client would essentially be just doing it ourselves because barely anything out of this merge is useful.

@soc1c
Copy link
Contributor Author

soc1c commented Jun 5, 2019

thanks, so let's not do it now.

@soc1c soc1c mentioned this issue Jun 10, 2019
Closed
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
ISS: Discussion Priority: 7 Low Type: Feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@austinabell @soc1c