-
Notifications
You must be signed in to change notification settings - Fork 13k
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 clippy #66207
Update clippy #66207
Conversation
@bors r+ p=1 |
📌 Commit c99091bb86ad1c69c28a294f322f743886c0d423 has been approved by |
Need another rustup here: rust-lang/rust-clippy#4792 |
Also you might need to update Cargo.lock. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors r- |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Please note that the clippy test still assume the test dependencies is in the host rlib directory. |
rust-lang/rust-clippy#4786 is merged, now you should reupdate clippy and set the TARGET_LIBS like this: |
1b8e8c1
to
c337fd0
Compare
@bors r+ p=2 |
📌 Commit c337fd02221eb436dbb0a5a0816b991cbd83f7e6 has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors r+ |
📌 Commit 4ba4552e838c851ff6c70e5eb2803195fdaaace6 has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
⌛ Testing commit 4ba4552e838c851ff6c70e5eb2803195fdaaace6 with merge bcad43d260220ae58e1e53956230e4565cf4a830... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
cc @pietroalbini something seems wrong there ^^ |
Also it took an hour and 30 minutes before failing. |
@bors retry i guess that server is just wonky |
⌛ Testing commit a498d2848ba7f3caf588c01a0a6ce4c84283100d with merge 6d07f23dbba0beb2b7ac6f23dc28d55e68e7386f... |
@bors r- |
Updated to clippy HEAD. @bors r+ retry |
📌 Commit 955f861 has been approved by |
☀️ Test successful - checks-azure |
📣 Toolstate changed by #66207! Tested on commit 2a9be46. 🎉 clippy-driver on windows: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). |
Tested on commit rust-lang/rust@2a9be46. Direct link to PR: <rust-lang/rust#66207> 🎉 clippy-driver on windows: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). 🎉 clippy-driver on linux: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra).
Fixes #66150
r? @ghost