-
Notifications
You must be signed in to change notification settings - Fork 625
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
Update to new nightly API #1268
Conversation
@cramertj are you thinking to rename |
@Nemo157 I've re-added it as |
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
Cargo.toml
Outdated
@@ -9,3 +9,6 @@ members = [ | |||
"futures-util", | |||
"futures-test", | |||
] | |||
|
|||
[patch.crates-io] | |||
pin-utils = { git = "https://github.com/Kroisse/pin-utils.git", branch = "pin" } |
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.
Do we plan on publishing the needed changes here?
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.
These are actually already published-- I just have to update the path here.
TODO: decide what to do w/ compat, executor trait.
Where should the executor trait go? I know @carllerche at least prefers an executor trait that is generic on the type of future it accepts, so I don't think the exact API is totally decided.
-util
seems wrong, but I hesitate to add another crate for it (especially since futures-executor is the good name, and that's already taken).Includes some changes from #1266 (thanks, @Kroisse!)