T. Todua
T. Todua
@carlosmiei whenever you have time to review this. thanks
tests passes too. @carlosmiei
why not? what's bad with exhaustive tests?
in advance, I can't say for sure. maybe if we see the frequent problem, maybe then relax it?
> As I said I have to check, because this is a trade-off as you know, the more exhaustive the tests over something we don't control are, higher the chance...
this PR is important, as it reveals exchange's implementational issues in trades (there had been revealed dozen of issues, which have been fixed in separate PRs till date). now, eventually,...
@roshsingh is it mandatory to have old nodejs version, which is no longer maintained ? https://nodejs.org/en/about/previous-releases does issue happen if you use latest LTS (supported) node version, and also latest...
this might be the commit you were referring to - https://github.com/ccxt/ccxt/commit/ce1189ee70a105c9c2922db75cb7aa323fa29168 I'm looking into this problem
@carlosmiei i will amend per your notes now, and lyk once i turn from draft into ready
Carlos, this PR is ready for review. (unrelated errors on travis)