-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[BackPort] Conservative updates #6655
Conversation
This reverts commit 4fb53e5
If we're updating a crate then its previous version in the lock file is no longer whitelisted. We may want to perhaps change this in the future!
When sources already exist in a `PackageRegistry` and the whitelist is updated in the package registry, be sure to update all the existing sources to ensure that everyone gets the same view of the intended whitelist.
r? @ehuss (rust_highfive has picked a reviewer for you, use r? to override) |
|
Looks like #6649 will be needed to get this to pass. I'll post it in a few minutes. |
Or you can push to this branch whatever is easier. |
The unused_imports warning is going to emit fixable suggestions in the near future, but that means parts of the cargo's test suite will break. This commit switches the tests to use the deprecated warning, which *shouldn't* be fixable at all.
@bors r+ I'm planning to update beta (on rust-lang/rust) towards the end of the week. |
📌 Commit 63412be has been approved by |
☀️ Test successful - checks-travis, status-appveyor |
This is a back port of #6611 as discussed in #6609