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.

Sign up for GitHub

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

Jump to bottom

chore: use box future #4307

Merged

fraidev merged 1 commit into infinyon:master from fraidev:use_box_future

Jan 2, 2025

Contributor

fraidev commented Dec 31, 2024 •

edited

Loading

I created a type for Pin<Box<dyn Future<_>> but I noticed now that there is BoxFuture that already does it.


          chore: use box future

a9c8202

fraidev requested a review from sehz

December 31, 2024 02:53

sehz approved these changes

View reviewed changes

fraidev added this pull request to the merge queue

github-merge-queue bot removed this pull request from the merge queue due to failed status checks

fraidev added this pull request to the merge queue

github-merge-queue bot removed this pull request from the merge queue due to failed status checks

fraidev added this pull request to the merge queue

github-merge-queue bot removed this pull request from the merge queue due to failed status checks

fraidev added this pull request to the merge queue

github-merge-queue bot removed this pull request from the merge queue due to failed status checks

fraidev added this pull request to the merge queue

Merged via the queue into infinyon:master with commit e8a0411

107 checks passed

fraidev deleted the use_box_future branch

January 2, 2025 20:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet