Repository navigation
unify Exchange.market_close signature to use "name" instead of "coin" #264
Description
not sure if this worth pursuing but Exchange.market_close uses https://github.com/hyperliquid-dex/hyperliquid-python-sdk/blob/master/hyperliquid/exchange.py#L244 uses "coin" instead of "name" but everywhere else in the code it uses name. I propose to change it to "name" as well
Activity
fix: unify Exchange.market_close signature to use 'name' instead of '…
KyaClaw commentedon Mar 2, 2026
I've created a pull request #266 to address this issue. The change renames the parameter from
cointonamein theExchange.market_closemethod, aligning it with the rest of the SDK's API consistency. Please review the PR when you have a moment.olzhang commentedon Mar 4, 2026
Thank you it looks good!
olzhang commentedon Mar 9, 2026
has this been merged? #266