Closed kieran-mackle closed 5 months ago
@bmoscon I used Black. Formatting changes reverted, ready for review.
formatting changes persist, will review once they have been resolved
@bmoscon Are there still formatting issues?
@bastienjalbert - there was a merge conflict - i resolved it but now the tests are failing. if @kieran-mackle can fix them I will merge it. I can take a look at fixing them myself but might not get time for a few days. likely the test data for bybit needs to be regenerated and then the expected callback counts will need to be updated in the test case. that may not be all though
Hey @kieran-mackle I would love to see this merged - do you think you could look into merge conflict @bmoscon mentioned? Please let me know if you could use a hand here, I would be more than happy to contribute 👍🏼
Hello @bmoscon, do you need help to finally merge this PR ?
@bastienjalbert the tests are failing, if they are fixed, I will merge it, otherwise at some point in the not too distant future I'll close this PR
@bmoscon I’ll take a look within next days/weeks to make unit test passing.
@bastienjalbert let me know if you are still working on this
@bmoscon Hard to find time lately. I’ll probably work on it, but in fews months.
@bastienjalbert feel free to fork off this PR and open a new one when you have time to take it on.
Description of code - what bug does this fix / what feature does this add?
Added support for spot websocket endpoints on Bybit. Current implementation includes trade and orderbook channels.
Testing
To connect to a spot endpoint, specify a standardised spot symbol. See example code below, connecting to both spot and perpetual endpoints for trades and orderbook.