kkrt-labs / kakarot-rpc

Kakarot ZK EVM Ethereum RPC adapter
MIT License
136 stars 103 forks source link

dev: rm `Arc` around `Arc<JsonRpcClient<HttpTransport>>` #1500

Open tcoratger opened 4 weeks ago

tcoratger commented 4 weeks ago

Describe the enhancement request

At the moment, we have a lots of Arc around JsonRpcClient<HttpTransport> like here:

https://github.com/kkrt-labs/kakarot-rpc/blob/443e148bce22d19e7455ffb1edc1faee5037d622/src/test_utils/katana/mod.rs#L89

Thanks to https://github.com/xJonathanLEI/starknet-rs/pull/666 and after the next release of starknet-rs we will be able to remove these Arc and just use a simple clone.

Gerson2102 commented 4 weeks ago

Hey, I can do this. When is no longer blocked.

tcoratger commented 4 weeks ago

Sure @Gerson2102 assigned