Skip to content
This repository has been archived by the owner on Feb 3, 2022. It is now read-only.

Update to RC6 #72

Merged
merged 12 commits into from
Aug 21, 2020
Merged

Update to RC6 #72

merged 12 commits into from
Aug 21, 2020

Conversation

JoshOrndorff
Copy link
Owner

@JoshOrndorff JoshOrndorff commented Aug 21, 2020

This PR updates the repo to use Substrate 2.0.0-rc6

There is a slight ugliness related to paritytech/substrate#6862 when calculating the type of can_author_with in the light client. A warning is thrown when compiling, and I've chosen not to suppress the warning as a reminder to fix this in the future.

node/src/service.rs Outdated Show resolved Hide resolved
FullClient, FullBackend, FullSelectChain,
BasicQueue<Block, TransactionFor<FullClient, Block>>,
sc_transaction_pool::FullPool<Block, FullClient>,
sc_consensus_pow::PowBlockImport<Block, Arc<FullClient>, FullClient, FullSelectChain, Sha3Algorithm<FullClient>, impl sp_consensus::CanAuthorWith<Block>>,
Copy link
Owner Author

@JoshOrndorff JoshOrndorff Aug 21, 2020

Choose a reason for hiding this comment

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

@dvdplm This is the type I ended up using thanks to @andresila 's help. I believe you were having a similar issue.

Copy link

@riusricardo riusricardo left a comment

Choose a reason for hiding this comment

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

The README needs some fixed/updates

README.md Outdated Show resolved Hide resolved
Copy link

@danforbes danforbes left a comment

Choose a reason for hiding this comment

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

Looks great!

@JoshOrndorff JoshOrndorff merged commit e6d1a02 into master Aug 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants