search

nautechsystems / nautilus_trader

A high-performance algorithmic trading platform and event-driven backtester
https://nautilustrader.io
GNU Lesser General Public License v3.0
1.71k stars 402 forks source link

Make `websocket` and `socket` disconnect sync function post atomic bool changes #1759

Closed twitu closed 4 days ago

twitu commented 5 days ago

Pull Request

Socket disconnection operation is now sync after moving to atomic bool in #1732.

Type of change

How has this change been tested?

pre-commit and rust tests were run

twitu commented 4 days ago

The existing implementation works. A later PR can improve the tests and unskip the skipped one.