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

Drop ClientProvider #5823

Merged
merged 3 commits into from
Apr 29, 2020
Merged

Drop ClientProvider #5823

merged 3 commits into from
Apr 29, 2020

Conversation

seunlanlege
Copy link
Contributor

Unfortunately Mega traits are hard to maintain.
Consumers should assemble the minimum required set of traits they need for client functionality.

@parity-cla-bot
Copy link

It looks like @seunlanlege signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@gnunicorn
Copy link
Contributor

test's failing.

Copy link
Contributor

@gnunicorn gnunicorn left a comment

Choose a reason for hiding this comment

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

thanks.

@gnunicorn
Copy link
Contributor

though I wonder if we want to keep it somewhere, like in the docs... 🤔

@seunlanlege
Copy link
Contributor Author

technically the polkadot companion PR is already merged 🤷‍♂️

@gnunicorn
Copy link
Contributor

@seunlanlege seems like the polkadot companion was incomplete – https://gitlab.parity.io/parity/substrate/-/jobs/486540 . Raise another companion PR please.

@seunlanlege
Copy link
Contributor Author

aii, both green paritytech/polkadot#1054 @gnunicorn

@gnunicorn gnunicorn merged commit 463e2bc into master Apr 29, 2020
@gnunicorn gnunicorn deleted the seun-remove-client-provider branch April 29, 2020 16:48
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.

3 participants