-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Correct some stability versions #42111
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
Conversation
These were found by running tidy on stable versions of rust and finding features stabilised with the wrong version numbers.
r? @brson (rust_highfive has picked a reviewer for you, use r? to override) |
impl_from! { u8, i16, #[stable(feature = "lossless_int_conv", since = "1.5.0")] } | ||
impl_from! { u8, i32, #[stable(feature = "lossless_int_conv", since = "1.5.0")] } | ||
impl_from! { u8, i64, #[stable(feature = "lossless_int_conv", since = "1.5.0")] } | ||
impl_from! { u8, i128, #[unstable(feature = "i128", issue = "35118")] } |
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.
Is this (and other changes) backwards compatible? I guess you could never use it before either?
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.
Yeah. Stability attributes don't have any effect on impls. The only visible change is in the docs, rustdoc displays the version numbers.
@bors r+ |
📌 Commit 2f703e4 has been approved by |
⌛ Testing commit 2f703e4 with merge 272e77f... |
Correct some stability versions These were found by running tidy on stable versions of rust and finding features stabilised with the wrong version numbers.
☀️ Test successful - status-appveyor, status-travis |
These were found by running tidy on stable versions of rust and finding
features stabilised with the wrong version numbers.