search

hyperliquid-dex / hyperliquid-python-sdk

SDK for Hyperliquid API trading with Python.
MIT License
238 stars 97 forks source link

Dev dependencies in prod dependencies. #14

Closed gaugau3000 closed 9 months ago

gaugau3000 commented 9 months ago

Hi,

Why is there dev dependencies in requirements.txt like pytest ?

Many tks.

gaugau3000 commented 9 months ago

Forget previous message and

Help to follow, transcript from discord.

gautier — Hier à 20:06 Oki, seems the requirements.txt is generated by Makefile when dev run make install. Not sure why the requirements.txt is for, but ok (difficult to identify source of truth) So pyproject.toml use poetry to install deps and pytest and pytest-recording shoudl not be in dependencies but in dev dependencie? ben_hl — Hier à 20:15 That sounds accurate. I haven't gotten a chance to try moving them and making sure everything still works. Should be able to get it done within the next day or two and release a new version of the sdk

see you.

traderben commented 9 months ago

This is now fixed in the latest version 0.1.19