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

Migrate the codebase to edition = "2018" ? #1867

Closed
Veetaha opened this issue Jul 29, 2020 · 3 comments
Closed

Migrate the codebase to edition = "2018" ? #1867

Veetaha opened this issue Jul 29, 2020 · 3 comments

Comments

@Veetaha
Copy link

Veetaha commented Jul 29, 2020

I wonder what is the main blocker for migrating the main codebase of serde crates to edition = "2018"?
IIRC it is not a breaking change to switch the edition:
https://doc.rust-lang.org/book/appendix-05-editions.html

All Rust compiler versions support any edition that existed prior to that compiler’s release, and they can link crates of any supported editions together. Edition changes only affect the way the compiler initially parses code. Therefore, if you’re using Rust 2015 and one of your dependencies uses Rust 2018, your project will compile and be able to use that dependency. The opposite situation, where your project uses Rust 2018 and a dependency uses Rust 2015, works as well.

I don't think we need to wait for 2.0 (if it is really going to happen at all, isn't it?) to update the edition the implementation of serde uses.

@MikailBag
Copy link

AFAIK serde supports Rust v1.13, which does not implement Rust 2018.

@Veetaha
Copy link
Author

Veetaha commented Jul 29, 2020

Might it make sense to update the msrv in a minor release?
For example this issue proposes to update the msrv in minor releases in tokio: tokio-rs/tokio#2718 (comment)

@dtolnay
Copy link
Member

dtolnay commented Jan 23, 2022

I don't think we need an issue open for this. There isn't any rush since I don't know of anything from 2018 edition that this crate needs, like new keywords or whatnot.

@dtolnay dtolnay closed this as completed Jan 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants