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

Conversation

@tomaka
Copy link
Contributor

@tomaka tomaka commented Jan 6, 2020

Performs a few clean-ups related to the sc-network-gossip crate:

  • Removes usage of sc_network::Context, and directly pass a &mut dyn Network instead.
  • Since gossiping doesn't use sc_network::Context anymore, removes the Context::send_consensus method. The Context is now used exclusively for the NetworkSpecialization stuff, which is hopefully going to disappear in the future as well.
  • Moves the Validator trait and its related traits and structs to a separate module.

@tomaka tomaka added the A0-please_review Pull request needs code review. label Jan 6, 2020
@tomaka tomaka requested a review from rphmeier January 6, 2020 14:10
@tomaka
Copy link
Contributor Author

tomaka commented Jan 6, 2020

Successfully tested on Kusama.

@tomaka
Copy link
Contributor Author

tomaka commented Jan 7, 2020

Polkadot PR: paritytech/polkadot#732


// You should have received a copy of the GNU General Public License
// along with Substrate. If not, see <http://www.gnu.org/licenses/>.

Copy link
Contributor

Choose a reason for hiding this comment

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

missing module doc

@gavofyork gavofyork merged commit 860b79b into paritytech:master Jan 9, 2020
@tomaka tomaka deleted the gossip-cleanup branch January 9, 2020 19:20
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants