-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Update clippy and RLS #49134
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 and RLS #49134
Conversation
Does it matter that Clippy's CI is failing?
|
clippy's CI fails on this because the change is not yet in nightly. I used a locally built rustc to confirm |
on that note. Is it possible to get the binaries from every PR? Maybe we could run clippy's CI on those instead of running it on nightly |
@oli-obk Yes, but those binaries will be deleted in 90 days. |
@bors r+ p=3 |
📌 Commit 5904543 has been approved by |
That's fine. I just want to download the most recent one for every CI execution |
@oli-obk It would be available at We reserve the right to change the URL though, so prefer to download through |
📌 Commit 8501e95 has been approved by |
⌛ Testing commit 8501e95 with merge 669f63f628c5f5ae71c6f23e7ca82a9ebddc9cb4... |
💔 Test failed - status-travis |
@bors retry rust-lang/rust-clippy@4edd140 was not yet merged into master. Created https://github.com/rust-lang-nursery/rust-clippy/tree/rustup-49134 for now. Please merge rust-lang/rust-clippy@4edd140 into master and delete the
|
Update clippy and RLS r? @Manishearth
☀️ Test successful - status-appveyor, status-travis |
Tested on commit rust-lang/rust@5508b27. Direct link to PR: <rust-lang/rust#49134> 🎉 clippy-driver on windows: build-fail → test-pass. 🎉 clippy-driver on linux: build-fail → test-pass. 🎉 rls on windows: build-fail → test-pass. 🎉 rls on linux: build-fail → test-pass.
r? @Manishearth