-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(mint-client): test parallel spends and reissuances #4407
chore(mint-client): test parallel spends and reissuances #4407
Conversation
00a32c9
to
bddd467
Compare
})); | ||
} | ||
|
||
let note_bags = futures::stream::iter(spend_tasks) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bags FTW!
I wonder why the linter didn't fail on the PR, will fix tomorrow. |
Landed after what your change is on top of, I guess. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@elsirion We land? |
bddd467
to
d5ce14f
Compare
37a9731
d5ce14f
to
37a9731
Compare
There actually was something to fix. |
@elsirion Still seems to be. :D |
37a9731
to
b939ec0
Compare
Yeah, forgot to run clippy with the extra command line options … |
b939ec0
to
970ecf1
Compare
Fixes #3988. Note that using
MemDatabase
still leads to deadlocks, but that's not production-ready anyway, so we can track it independently.