-
Notifications
You must be signed in to change notification settings - Fork 326
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 MSRV to 1.56 #1516
Update MSRV to 1.56 #1516
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.
I don't understand all the changes, in particular around rust-toolchain.toml
and the edition. Approving but copying @soareschen since he may have more informed opinions on reviewing this!
Just a question for my understanding: It seems edition 2018 still works, is it advisable (or good practice) to change to 2021?
In this commit (b6d6bdb) we updated to 1.56, but left edition unchanged, and it compiled.
Since the minimum Rust version we support is 1.56, we can now opt-in into the 2021 edition which unlocks some new features: https://blog.rust-lang.org/2021/10/21/Rust-1.56.0.html#rust-2021 The |
* Switch to `rust-toolchain`.toml * Add `rust-version` field to all packages to specify MSRV * Enable all features on docs.rs for modules and relayer libraries * Update MSRV in README * Update edition to 2021 * Add `TryFrom`, `TryInto` and `FromIterator` to the prelude to match the 2021 std prelude * Add changelog entry
Closes: #1519
Note: MSRV = Minimum Supported Rust Version
For contributor use:
unclog
.docs/
) and code comments.Files changed
in the Github PR explorer.