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

Add Rococo test network #1363

Merged
45 commits merged into from
Aug 4, 2020
Merged

Add Rococo test network #1363

45 commits merged into from
Aug 4, 2020

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jul 6, 2020

No description provided.

@bkchr bkchr added A0-please_review Pull request needs code review. B2-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. labels Jul 6, 2020
@bkchr bkchr requested a review from ddorgan July 6, 2020 15:02
pub MinSolutionScoreBump: Perbill = Perbill::from_rational_approximation(5u32, 10_000);
}

impl staking::Trait for Runtime {
Copy link
Member

Choose a reason for hiding this comment

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

any need for staking? why not just a poa chain?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point.

@cecton could you please remove the staking pallet and regenerate the chain specs?

Copy link
Contributor

Choose a reason for hiding this comment

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

To regenerate the chain specs I use build-spec --chain=rococo and I remove the entries in bootbodes in the JSON?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind it's fine I think. Can you double check?

@cecton cecton requested a review from gavofyork July 13, 2020 10:44
@cecton
Copy link
Contributor

cecton commented Jul 13, 2020

We should merge if the reviewers are ok

@cecton
Copy link
Contributor

cecton commented Jul 27, 2020

I have no idea what cargo is doing but it's using a different substrate commit (8180062 - latest) than the one in the Cargo.lock (e824e8a)... this makes the update harder. Working on this now...

@cecton
Copy link
Contributor

cecton commented Jul 31, 2020

🤣

service/src/lib.rs Outdated Show resolved Hide resolved
runtime/rococo/src/lib.rs Outdated Show resolved Hide resolved
bkchr and others added 2 commits August 3, 2020 21:49
Co-authored-by: Robert Habermeier <rphmeier@gmail.com>
Co-authored-by: Robert Habermeier <rphmeier@gmail.com>
@rphmeier
Copy link
Contributor

rphmeier commented Aug 4, 2020

bot merge

@ghost
Copy link

ghost commented Aug 4, 2020

Trying merge.

@ghost ghost merged commit 5fd6440 into master Aug 4, 2020
@ghost ghost deleted the bkchr-rococo branch August 4, 2020 09:41
ordian added a commit that referenced this pull request Aug 5, 2020
* master:
  Unalias Substrate Imports (#1530)
  Rewrite client handling (#1531)
  Integrate ChainApi with all messages (#1533)
  Add Rococo test network (#1363)
  Fix a typo parathreads -> parachains (#1529)
  Cleanup upcoming paras (#1527)
  Sudo wrapper for paras (#1517)
  Implementer's guide: notes on contextual execution (#1525)
  Companion for substrate/6782 (#1523)
  Sort out validation errors (#1516)
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants