-
Notifications
You must be signed in to change notification settings - Fork 65
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 tokio version 1.0 #111
Conversation
Also, something that I forgot to mention, the rt and macros features of tokio were moved to dev-dependencies, they just weren't needed. |
@QuietMisdreavus could you please give this a review too? |
This requires a custom fork/branch of egg-mode until egg-mode-rs/egg-mode#111 is merged. This PR just confirms that the code is running with tokio 1.0.2 but will probably not merge to main until egg-mode is updated. Closes #5.
any updates on this? I am currently using [replace]
"egg-mode:0.15.0" = { git = "https://github.com/RAnders00/egg-mode", branch = "tokio-v1.0-upgrade", rev = "11dc221533288ebcb1b135c2df65f187a80cd255", features = ["rustls"], default-features = false } as a hoffix and it works, but dropping this would be great! |
Thanks! Maybe @adwhit has time to look at this? I saw them in the commit history for the most recent git commit. |
Sure, I can take a look at this. It's obviously something we want |
LGTM, thanks! |
This requires a custom fork/branch of egg-mode until egg-mode-rs/egg-mode#111 is merged. This PR just confirms that the code is running with tokio 1.0.2 but will probably not merge to main until egg-mode is updated. Closes #5.
This requires a custom fork/branch of egg-mode until egg-mode-rs/egg-mode#111 is merged. This PR just confirms that the code is running with tokio 1.0.2 but will probably not merge to main until egg-mode is updated. Closes #5.
My pull request generally bumps all outdated dependencies, but most importantly it updates to tokio v1.0.