-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Allow some deprecated items #2117
Conversation
This comment has been minimized.
This comment has been minimized.
|
These were deprecated recently but we don't want to use them yet to not break compatibility with older rustc versions unnecessary. Explicitly allowing them should fix the build for nightly with `#[deny(warnings)]`.
2a62fae
to
3421d4c
Compare
Rebased on top of your branch, @JohnTitor, so it now includes the |
Hmm, an error unrelated to this PR has occurred in |
Yeah, the tests even passed but
when running rustdoc. I'll read this as "CI is green" :D |
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.
lgtm, nice work!
Given that many folks on the unsafe code guidelines feel that |
The problem with using |
Our policy is that we only bump if we have a good reason (e.g. not just for a convenience function). I think that meets this bar. We usually want to have some amount of lag so folks using their distro's Rust have a chance of being able to get the right version, but I think right now we're at least a few months from 2.0 shipping so it's fine. |
Yeah, it seems we have a good reason to bump up the version. |
👍 |
These were deprecated recently but we don't want to use them yet to not
break compatibility with older rustc versions unnecessary. Explicitly
allowing them should fix the build for nightly with
#[deny(warnings)]
.