-
Notifications
You must be signed in to change notification settings - Fork 435
Update eth-abi dependency #40
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
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! |
|
I published a new version that drops support for 3.7 |
|
#60 upgraded the eth-abi dependency, so I'm going to close this PR. Feel free to open a new one if there's a compelling reason to upgrade further |
This pull request updates the
eth-abidependency in thehyperliquid-python-sdkproject. The changes are as follows:eth-abiversion inpyproject.tomlto support both Python 3.7 and Python 3.8+.eth-abiupdate.This update ensures that hyperliquid SDK stays up-to-date with the latest
eth-abiversion, improving its compatibility and performance with Ethereum-based operations.Please review the changes and let me know if there are any concerns or suggestions.