-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update eth-abi dependency #40
base: master
Are you sure you want to change the base?
Conversation
Also getting conflicts with other Eth based development applications.
|
I think |
Sorry it took me so long to review this, but I don't think this works on python 3.7 To test it I built a wheel: I don't see why this change would cause this error, but I don't have an issue with following the same steps to install the wheel without this change |
Python 3.7 is end of life and we have very little 3.7 usage: https://pypistats.org/packages/hyperliquid-python-sdk |
@traderben I think it would be a good idea to drop the 3.7 version. Let me know when you plan to do it, I could help upgrade the other dependencies in case! |
This pull request updates the
eth-abi
dependency in thehyperliquid-python-sdk
project. The changes are as follows:eth-abi
version inpyproject.toml
to support both Python 3.7 and Python 3.8+.eth-abi
update.This update ensures that hyperliquid SDK stays up-to-date with the latest
eth-abi
version, improving its compatibility and performance with Ethereum-based operations.Please review the changes and let me know if there are any concerns or suggestions.