zksync-sdk / zksync-python

Python 3.8 SDK for zkSync
MIT License
116 stars 64 forks source link

Make `EthereumSignerInterface` sign methods synchronous #11

Closed hukkin closed 3 years ago

hukkin commented 3 years ago

These interface being async doesn't seem to achieve anything, but increases API complexity, so I'd suggest making it synchronous.

It all boils down to EthereumSignerWeb3.sign which is an async def function but doesnt await anywhere.

This is also consistent with ZkSyncSigner where the sign function is synchronous.