JoshOrndorff / utxo-workshop

A Substrate UTXO workshop
The Unlicense
117 stars 81 forks source link

Convert UTXO node to PoW #59

Closed JoshOrndorff closed 4 years ago

JoshOrndorff commented 4 years ago

This PR replaces the Aura/Grandpa consensus with PoW consensus, and also takes first steps toward bitcoin-ifying the chain in other ways.

nczhu commented 4 years ago

I'm on the latest commit: 17ce57c5fb5e7fa39ea1cba9e0a354b063ec5b1e

Demo works, but tests are broken. I'm getting the following error. @JoshOrndorff if you can fix this and get tests running, then we're ready to merge!

Nicoles-MBP-2:utxo-workshop nicole$ cargo test -p utxo-runtime
   Compiling utxo-runtime v2.0.0-alpha.5 (/Users/nicole/iCloud Drive (Archive)/Desktop/utxo-workshop/runtime)
   Compiling sp-consensus-pow v0.8.0-alpha.5
error[E0046]: not all trait items implemented, missing: `BlockAuthor`, `Issuance`
   --> runtime/src/utxo.rs:326:2
    |
27  |     type BlockAuthor: BlockAuthor;
    |     ------------------------------ `BlockAuthor` from trait
...
30  |     type Issuance: Issuance<<Self as system::Trait>::BlockNumber, Value>;
    |     --------------------------------------------------------------------- `Issuance` from trait
...
326 |     impl Trait for Test {
    |     ^^^^^^^^^^^^^^^^^^^ missing `BlockAuthor`, `Issuance` in implementation

error: aborting due to previous error

For more information about this error, try `rustc --explain E0046`.
error: could not compile `utxo-runtime`.
warning: build failed, waiting for other jobs to finish...
error: build failed
nczhu commented 4 years ago

nvm my latest commit fixes the issue. @JoshOrndorff you are ready / welcome to merge !