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