Debnil Sur

Results 7 comments of Debnil Sur

@nikhilsaraf: Let's go over this PR in our pair programming session quickly. I have zero idea why this isn't working and how to fix it. What happens is that when...

@nikhilsaraf WIP PR for trade metrics. The actual metrics are not added, since we change how metrics are added in the existing PRs, but all the functions to generate those...

@nikhilsaraf: awesome! I've done the bulk of this review, thanks so much. The only part that's left is the merging trade metrics with map. This only makes sense to do...

@nikhilsaraf : most of the requested changes are made, but there's now an import cycle. `plugins/tradeMetricsHandler` imports `models.Trade`, which imports `plugins` indirectly. I don't think there's a way around the...

@daxpedda: I'd like to use `rust-fork` to test an `async` function using `tokio`, and this PR looks perfect for that. Are there any plans to merge this? If you need...

Wow - thanks so much for the fast turnaround and excellent work! @AltSysrq: this pull request, along with #10, would extend support for async tests. Merging both would be incredibly...

@AltSysrq: thanks for all your work on this crate, it's really excellent. If it could be extended to support async functions, it would be even more excellent. Could this pull...