-
Notifications
You must be signed in to change notification settings - Fork 27k
feat(core): Add async run method on ExperimentalPendingTasks
#56546
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
Conversation
thePunderWoman
left a comment
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.
reviewed-for: public-api
0ae0426 to
3f64297
Compare
3f64297 to
e37fb54
Compare
This helper method is simply a convenience function that reduces some boilerplate with manually adding and removing a task around some asynchronous function.
e37fb54 to
92d9302
Compare
AndrewKushnir
left a comment
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.
Looks great, thanks @atscott 👍
AndrewKushnir
left a comment
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.
Reviewed-for: public-api
|
This PR was merged into the repository by commit 3ebe6b4. The changes were merged into the following branches: main |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This helper method is simply a convenience function that reduces some boilerplate with manually adding and removing a task around some asynchronous function.