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

[WIP do not merge] Extract sc-network-sync crate #10514

Closed
wants to merge 2 commits into from

Conversation

wigy-opensource-developer
Copy link
Contributor

@wigy-opensource-developer wigy-opensource-developer commented Dec 20, 2021

Extract synchronization related code into its own crate. This is the last protocol that was handled in a special way in the network crate.

closes: #8686
polkadot companion: paritytech/polkadot#4701

@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label Dec 20, 2021
@wigy-opensource-developer wigy-opensource-developer changed the title Extract sc-network-sync crate [WIP do not merge] Extract sc-network-sync crate Jan 12, 2022
@wigy-opensource-developer wigy-opensource-developer added B5-clientnoteworthy B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit and removed B5-clientnoteworthy labels Jan 12, 2022
@wigy-opensource-developer wigy-opensource-developer marked this pull request as ready for review January 12, 2022 13:52
@github-actions github-actions bot added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Jan 12, 2022
@wigy-opensource-developer
Copy link
Contributor Author

This is my first PR with a companion, so I wanted to test it on CI. It is not ready for review yet, but draft PRs skip running CI.

@stale
Copy link

stale bot commented Feb 11, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Feb 11, 2022
@wigy-opensource-developer
Copy link
Contributor Author

Yip, still relevant.

@stale stale bot removed the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Feb 11, 2022
@stale
Copy link

stale bot commented Mar 13, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Mar 13, 2022
@wigy-opensource-developer
Copy link
Contributor Author

Yeah. Hi, bot! I was distracted by other issues, but I am getting back to this.

@stale stale bot removed the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Mar 13, 2022
@stale
Copy link

stale bot commented Apr 12, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Apr 12, 2022
@stale stale bot closed this Apr 27, 2022
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. A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extracting the syncing protocol from sc-network
1 participant