-
Notifications
You must be signed in to change notification settings - Fork 98
Conversation
I know this is not something we keep an eye on, but what's the MSRV of that change? |
Dumb question: MSRV is The "minimum supported Rust version" right? "increments are a breaking Semver change" policy may applies to this repository 🤔 should it?
AFAIK minimal stable toolchain version with rust edition 2021 is rustc 1.56.0 This could be propagated in all change logs and maybe include a version bump. But what type? patch, minor or major? |
Changing the edition isn't a breaking change because different edition crates can depend on each other. However, this also bumps the required Rust version. I think we should apply these two changes to this PR:
|
from Will, this is what srtool is using and is tested. |
Not sure how this is important because that is only a security fix and nothing that interacts with compatibility. |
edition = "2021" rust-version = "1.56.1"
For reference, I tried to get this to
With |
My post stated "1.56" and not "1.56.0". The fix version is just noise here. |
Based on a quick DM, it was decided this PR should be merged 🤝 |
No description provided.