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

Bump Substrate#816

Merged
gavofyork merged 31 commits intomasterfrom
gav-upsub
Feb 9, 2020
Merged

Bump Substrate#816
gavofyork merged 31 commits intomasterfrom
gav-upsub

Conversation

@gavofyork
Copy link
Member

@gavofyork gavofyork commented Feb 1, 2020

Brings it up to date with the latest Substrate.

@gavofyork gavofyork added the A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). label Feb 1, 2020
@gavofyork gavofyork added A0-please_review Pull request needs code review. and removed A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). labels Feb 1, 2020
@gavofyork gavofyork added the B0-silent Changes should not be mentioned in any release notes label Feb 4, 2020
@cecton cecton mentioned this pull request Feb 5, 2020
Copy link
Contributor

@cecton cecton left a comment

Choose a reason for hiding this comment

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

LGTM

@gavofyork gavofyork merged commit 6a2092d into master Feb 9, 2020

let exit = inner_exit_2.clone();
tokio::spawn(future::select(res.boxed(), exit));
tokio::spawn(res.boxed());
Copy link
Member

Choose a reason for hiding this comment

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

@cecton did you check that the node still exits?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes there is a test for that but in the other branch. I will make another PR to add it now

Copy link
Member

Choose a reason for hiding this comment

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

Okay, ty!

tomusdrw pushed a commit that referenced this pull request Mar 26, 2021
* more tests for finality relay

* clippy

* remove env_logger dep

* fmt

* more clippy

* removed prune_unjustified_headers

* review
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. B0-silent Changes should not be mentioned in any release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants