-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 sync_wrapper
to 1.0.0 from 0.1.1
#2667
Conversation
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.
Thanks!
Do you mind updating the MSRV in |
Damn, more errors. If you want you can try to fix them here, otherwise I'll try to find some time for doing that separately soon. |
Sorry, I could not fix CI/test-nightly. I fix only CI/check. |
I think I have fixed all the Cl/test-nightly. |
Wow that was a lot of stuff that created new warnings all at once. Thanks a lot for taking care of all the small changes required to make CI happy again! |
Motivation
sync_wrapper
had a dependencyfuture-core
until version 0.1.1.Now,
future-core
is optional insync_wrapper
from version 0.1.2struct SyncStream
is dependentfuture-core
butaxum
doesn't usestruct SyncStream
.Updating to version 0.1.2 or later separates
sync_wrapper
fromfuture-core
.Solution
Update
sync_wrapper
to 1.0.0