Pull out LedgerClient and execute function?

[mattyg]

I'm wondering it makes sense to keep this crate purely for typecasting between alloy and ethers, and pull out the functionality under LedgerClient and ExecutableTransaction.execute into another crate. Maybe keep both crates in this repo as the abstraction layer for working with alloy / ethers to simplify the transition between them?

@hardyjosh @thedavidmeister

[hardyjosh]

I was already hoping to use it in rain.interpreter actually - I was working on adding a ReadableTransction in this repo.

Why did you want to move it out?

[mattyg]

Why did you want to move it out?

just thinking of separating concerns: keep one crate purely for the ethers <-> alloy typecasting, and make another for the higher-level abstractions for using ethers/alloy in real-world settings (i.e setting up a ledger client, executing a transaction with feedback logs, etc.)

[hardyjosh]

Does make sense actually. The reason we put it here is because it's an alloy interface to what is ethers under the hood. Once it becomes all alloy we'll still want it separated but it won't have an ethers component anymore. Unless @thedavidmeister disagrees I think go ahead and separate out.