airgap-it / tezos-rust-sdk

MIT License
20 stars 10 forks source link

Removes the tight coupling between the RPC client and `Contract` #29

Closed emchristiansen closed 1 year ago

emchristiansen commented 1 year ago

I just found this library and started to play with it - very nice!

My application would require loose coupling between the RPC client and the Contract struct as I need to be able to easily switch RPC clients. Thus I've refactored tezos-contract so that the user provides the RPC client as needed, rather than having it inside the relevant structs.

What do you think?

P.S. I also added a .rustfmt.toml (empty) so that the formatter respects your formatting preferences, even when this project is a git submodule of another project.

RomarQ commented 1 year ago

The changes make sense to me. The breaking changes should also not be a problem yet.

@godenzim Let me know if you also agree with the purposed changes.