fixup! refactor: _

Author: valeratrades

examples/binance/market.rs

@@ -8,15 +8,13 @@ async fn main() {
 	color_eyre::install().unwrap();
 	v_utils::utils::init_subscriber(v_utils::utils::LogDestination::xdg("v_exchanges"));
 
-	let mut b = Binance::new();
+	let mut b = Binance::default();
 
 	b.update_default_option(BinanceOption::HttpUrl(BinanceHttpUrl::FuturesUsdM));
 
 	//before implementing the trait for bybit too, was able to just do eg: `let klines = client.futures_klines(("BTC", "USDT").into(), "1m".into(), 2, None, None).await.unwrap();`
 
-	let klines = b.futures_klines(("BTC", "USDT").into(), "1m".into(), 2, None, None)
-		.await
-		.unwrap();
+	let klines = b.futures_klines(("BTC", "USDT").into(), "1m".into(), 2, None, None).await.unwrap();
 	let price = b.futures_price(("BTC", "USDT").into()).await.unwrap();
 	dbg!(&klines, price);
 

examples/bybit/market.rs

@@ -8,12 +8,12 @@ async fn main() {
 	color_eyre::install().unwrap();
 	v_utils::utils::init_subscriber(v_utils::utils::LogDestination::xdg("v_exchanges"));
 
-	let mut bb = Bybit::new();
+	let mut bb = Bybit::default();
 
-	let ticker: serde_json::Value = bb.0
-		.get("/v5/market/tickers", &[("category", "spot"), ("symbol", "BTCUSDT")], [BybitOption::Default])
-		.await
-		.expect("failed to get ticker");
+	let ticker: serde_json::Value =
+		bb.0.get("/v5/market/tickers", &[("category", "spot"), ("symbol", "BTCUSDT")], [BybitOption::Default])
+			.await
+			.expect("failed to get ticker");
 	println!("Ticker:\n{ticker}");
 
 	if let (Ok(key), Ok(secret)) = (env::var("BYBIT_TIGER_READ_KEY"), env::var("BYBIT_TIGER_READ_SECRET")) {

v_exchanges/src/binance/mod.rs

@@ -9,7 +9,7 @@ use v_utils::{
 
 use crate::core::{AssetBalance, Exchange, Klines};
 
-#[derive(Clone, Debug, Default, Deref, DerefMut, WrapNew)]
+#[derive(Clone, Debug, Default, Deref, DerefMut)]
 pub struct Binance(pub Client);
 
 //? currently client ends up importing this from crate::binance, but could it be possible to lift the [Client] reexport up, and still have the ability to call all exchange methods right on it?

v_exchanges/src/bybit/mod.rs

@@ -1,19 +1,18 @@
 pub mod account;
 
 use color_eyre::eyre::Result;
-use derive_more::derive::Deref;
-use derive_more::derive::DerefMut;
-use v_exchanges_adapters::Client;
-use v_exchanges_adapters::bybit;
-use v_utils::macros::WrapNew;
-use v_utils::trades::{Asset, Pair, Timeframe};
+use derive_more::derive::{Deref, DerefMut};
+use v_exchanges_adapters::{Client, bybit};
+use v_utils::{
+	macros::WrapNew,
+	trades::{Asset, Pair, Timeframe},
+};
 
 use crate::core::{AssetBalance, Exchange, Klines};
 
-#[derive(Clone, Debug, Default, Deref, DerefMut, WrapNew)]
+#[derive(Clone, Debug, Default, Deref)]
 pub struct Bybit(pub Client);
 
-
 //? currently client ends up importing this from crate::binance, but could it be possible to lift the [Client] reexport up, and still have the ability to call all exchange methods right on it?
 impl Exchange for Bybit {
 	async fn futures_klines(&self, symbol: Pair, tf: Timeframe, limit: u32, start_time: Option<u64>, end_time: Option<u64>) -> Result<Klines> {

v_exchanges_adapters/src/lib.rs

@@ -39,11 +39,6 @@ pub struct Client {
 }
 
 impl Client {
-	/// Creates a new [Client].
-	pub fn new() -> Self {
-		Self::default()
-	}
-
 	/// Update the default options for this [Client]
 	#[inline(always)]
 	pub fn update_default_option<O>(&mut self, option: O)

v_exchanges_api_generics/src/websocket.rs

@@ -1,3 +1,4 @@
+use core::slice::SlicePattern;
 use std::{
 	collections::hash_map::{Entry, HashMap},
 	mem,
@@ -7,11 +8,11 @@ use std::{
 	},
 	time::Duration,
 };
+
 use futures_util::{
 	sink::SinkExt,
 	stream::{SplitSink, StreamExt},
 };
-use core::slice::SlicePattern;
 use parking_lot::Mutex as SyncMutex;
 use tokio::{
 	net::TcpStream,