Skip to content
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

Merged
merged 2 commits into from
Mar 18, 2018
Merged

Update clippy and RLS #49134

merged 2 commits into from
Mar 18, 2018

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Mar 18, 2018

@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

Does it matter that Clippy's CI is failing?

error[E0308]: mismatched types
  --> clippy_lints/src/unsafe_removed_from_name.rs:46:29
   |
46 |         UseTreeKind::Simple(Some(new_name)) => {
   |                             ^^^^^^^^^^^^^^ expected struct `syntax::ast::Ident`, found enum `std::option::Option`
   |
   = note: expected type `syntax::ast::Ident`
              found type `std::option::Option<_>`
error[E0308]: mismatched types
  --> clippy_lints/src/unsafe_removed_from_name.rs:55:29
   |
55 |         UseTreeKind::Simple(None) |
   |                             ^^^^ expected struct `syntax::ast::Ident`, found enum `std::option::Option`
   |
   = note: expected type `syntax::ast::Ident`
              found type `std::option::Option<_>`
error: aborting due to 2 previous errors

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 18, 2018
@oli-obk
Copy link
Contributor Author

oli-obk commented Mar 18, 2018

clippy's CI fails on this because the change is not yet in nightly. I used a locally built rustc to confirm

@oli-obk
Copy link
Contributor Author

oli-obk commented Mar 18, 2018

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

@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

@oli-obk Yes, but those binaries will be deleted in 90 days.

@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

@bors r+ p=3

@bors
Copy link
Contributor

bors commented Mar 18, 2018

📌 Commit 5904543 has been approved by kennytm

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 18, 2018
@oli-obk
Copy link
Contributor Author

oli-obk commented Mar 18, 2018

Yes, but those binaries will be deleted in 90 days.

That's fine. I just want to download the most recent one for every CI execution

@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

@oli-obk It would be available at https://s3-us-west-1.amazonaws.com/rust-lang-ci2/rustc-builds/$COMMIT/$COMPONENT-nightly-$TARGET.tar.xz where $COMPONENT is rustc, rust-std, cargo, etc.

We reserve the right to change the URL though, so prefer to download through rustup-toolchain-install-master or cargo-bisect-rustc.

@kennytm kennytm mentioned this pull request Mar 18, 2018
@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

@bors r+

Sorry for hijack. I've included an update to RLS here since it was broken by #48986.

@bors
Copy link
Contributor

bors commented Mar 18, 2018

📌 Commit 8501e95 has been approved by kennytm

@kennytm kennytm changed the title Update clippy Update clippy and RLS Mar 18, 2018
@bors
Copy link
Contributor

bors commented Mar 18, 2018

⌛ Testing commit 8501e95 with merge 669f63f628c5f5ae71c6f23e7ca82a9ebddc9cb4...

@bors
Copy link
Contributor

bors commented Mar 18, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 18, 2018
@kennytm
Copy link
Member

kennytm commented Mar 18, 2018

@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 rustup-49134 branch after this is merged and a new nightly is released.

[00:01:11] Submodule 'src/tools/clippy' (https://github.com/rust-lang-nursery/rust-clippy.git) unregistered for path 'src/tools/clippy'
[00:01:11] Submodule 'src/tools/clippy' (https://github.com/rust-lang-nursery/rust-clippy.git) registered for path 'src/tools/clippy'
[00:01:11] error: Server does not allow request for unadvertised object 4edd140e57cce900fa930e1439bab469f5bbce46
[00:01:11] Fetched in submodule path 'src/tools/clippy', but it did not contain 4edd140e57cce900fa930e1439bab469f5bbce46. Direct fetching of that commit failed.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 18, 2018
@bors
Copy link
Contributor

bors commented Mar 18, 2018

⌛ Testing commit 8501e95 with merge 5508b27...

bors added a commit that referenced this pull request Mar 18, 2018
@bors
Copy link
Contributor

bors commented Mar 18, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 5508b27 to master...

@bors bors merged commit 8501e95 into rust-lang:master Mar 18, 2018
@kennytm-githubbot
Copy link

📣 Toolstate changed by #49134!

Tested on commit 5508b27.
Direct link to PR: #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.

kennytm-githubbot added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Mar 18, 2018
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants