Skip to content
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

Support AsyncSystem::all on void Futures. #899

Merged
merged 5 commits into from
Sep 30, 2024
Merged

Conversation

kring
Copy link
Member

@kring kring commented Jun 4, 2024

Fixes #898

CC @jaadelgren

@jaadelgren
Copy link

That was quick, thanks!

@j9liu j9liu modified the milestone: July Release Jun 24, 2024
@j9liu j9liu added this to the August 2024 Release milestone Jul 23, 2024
@kring
Copy link
Member Author

kring commented Jul 31, 2024

Moving this out of tomorrow's release because our capacity for reviews before then is low, and because it's not user-visible anyway.

@kring kring removed this from the August 2024 Release milestone Jul 31, 2024
@kring kring added this to the October 2024 Release milestone Sep 9, 2024
@kring kring merged commit c29f56b into main Sep 30, 2024
24 checks passed
@kring kring deleted the all-with-void-futures branch September 30, 2024 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AsyncSystem::all() does not support void futures
4 participants