From 9450b5409eac48c77d765bf92bc34c4d90958748 Mon Sep 17 00:00:00 2001
From: Andreas Doerr <adoerr@users.noreply.github.com>
Date: Thu, 25 Mar 2021 10:38:41 +0100
Subject: [PATCH] Make Client convenience trait public (#126)

---
 client/beefy/src/lib.rs | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/client/beefy/src/lib.rs b/client/beefy/src/lib.rs
index 467f9edc22d1f..9b2917082ae11 100644
--- a/client/beefy/src/lib.rs
+++ b/client/beefy/src/lib.rs
@@ -62,7 +62,7 @@ pub fn beefy_peers_set_config() -> sc_network::config::NonDefaultSetConfig {
 /// has to satisfy. Ideally that should actually be a trait alias. Unfortunately as
 /// of today, Rust does not allow a type alias to be used as a trait bound. Tracking
 /// issue is <https://github.com/rust-lang/rust/issues/41517>.
-pub(crate) trait Client<B, BE, P>:
+pub trait Client<B, BE, P>:
 	BlockchainEvents<B> + HeaderBackend<B> + Finalizer<B, BE> + ProvideRuntimeApi<B> + Send + Sync
 where
 	B: Block,
@@ -107,9 +107,7 @@ where
 
 /// Start the BEEFY gadget.
 ///
-/// This is a thin shim around running and awaiting a BEEFY worker. The [`Client`]
-/// convenience trait is not used here on purpose. We don't want to leak it into the
-/// public interface of the BEEFY gadget.
+/// This is a thin shim around running and awaiting a BEEFY worker.
 pub async fn start_beefy_gadget<B, P, BE, C, N, SO>(
 	client: Arc<C>,
 	key_store: SyncCryptoStorePtr,
@@ -123,7 +121,7 @@ pub async fn start_beefy_gadget<B, P, BE, C, N, SO>(
 	P::Public: AppPublic + Codec,
 	P::Signature: Clone + Codec + Debug + PartialEq + TryFrom<Vec<u8>>,
 	BE: Backend<B>,
-	C: BlockchainEvents<B> + HeaderBackend<B> + Finalizer<B, BE> + ProvideRuntimeApi<B> + Send + Sync,
+	C: Client<B, BE, P>,
 	C::Api: BeefyApi<B, P::Public>,
 	N: GossipNetwork<B> + Clone + Send + 'static,
 	SO: SyncOracleT + Send + 'static,