Skip to content
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

Move edition to 2021 #688

Merged
merged 3 commits into from Jul 29, 2022

Conversation

chetaldrich
Copy link
Contributor

@chetaldrich chetaldrich commented Jul 24, 2022

Looks like since I've last looked at the repo the update to
MSRV 1.56 happened and we don't have any clippy errors on the 2021
edition. Therefore, I think we can just up the edition now.
If there's a good reason not to do so let me know and I can close this.

I've also set the MSRV explicitly in the Cargo.toml so that users that happen to have an older rust version set (like I did when I was testing 1.51 at some point) get an error telling them they're on the wrong version instead of abstruse compiler errors.

Closes #553, also closes #595 since 1.51 will no longer be supported.

chetaldrich and others added 3 commits July 24, 2022 12:49
Looks like since I've last looked at the repo, the update to
MSRV 1.56 happened, and we don't have any clippy errors on the 2021
edition, so I _think_ we can just up this value now.

Closes tokio-rs#553
This was introduced in [Rust 1.56](https://blog.rust-lang.org/2021/10/21/Rust-1.56.0.html#cargo-rust-version)
so we can set this explicitly here. This means that folks that happen to have an older
rust version set don't run into abstruse compiler errors.
Copy link
Member

@LucioFranco LucioFranco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@chetaldrich
Copy link
Contributor Author

no problem, thanks for taking a look!

@LucioFranco LucioFranco merged commit 11b5406 into tokio-rs:master Jul 29, 2022
@chetaldrich chetaldrich deleted the chet-upgrade-edition-2021 branch July 29, 2022 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix warnings with 1.51 Fix clippy warnings introduced with 2021 edition
2 participants