submodules: update rls and clippy#57805
Conversation
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
|
cc @Xanewok |
|
Thanks! @bors r+ p=1 |
|
📌 Commit 26cbe198e63a9201c83601693fe02c1ca2f67f9a has been approved by |
|
⌛ Testing commit 26cbe198e63a9201c83601693fe02c1ca2f67f9a with merge 39506b3b0d74d7178d6bd2d11de5e1a0e230483f... |
|
💔 Test failed - status-appveyor |
|
This submodule update pulls Tokio and some related bits - that's surely why we depend on more winapi features. @matthiaskrgr From what I understand is the problem, could you please add the @bors delegate+ |
|
✌️ @matthiaskrgr can now approve this pull request |
|
@Xanewok This needs to go into the rls repo, right? |
|
@matthiaskrgr whoops, I was sure I linked to the list: rust/src/tools/rustc-workspace-hack/Cargo.toml Lines 22 to 53 in 7164a9f That's in-tree, so it doesn't need an extra PR against RLS. |
|
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 |
|
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 |
|
clippy broke again... |
Yeah, although watch out since there are also unrelated changes including recently merged Iterator::is_sorted 😅 |
Changes: ```` Remove state.analysis due to Rust PR rust-lang#57476 Improve missing nightly readme info Bump languageserver-types to v0.54.0 and renam crate name to lsp-types Delete bors.toml Fix tests Fix rust-lang/rls#1231 Implement asynchronous message reading Use typed requests Implement Tokio-based test LSP client Update README.md to account for Travis url change Simplify wait_for_all recv calls Update dependencies Revert NLL bug workaround Remove old test_data entry in .gitignore Reorganize some tests Don't test RLS binary target directly Move tooltip tests to integration tests Simplify tooltip test harness Only use FIXTURES_DIR to determine fixtures Remove src/test/mod.rs Centralise FIXTURES_DIR across unit and integration tests Move lens test to tests/ Suppress unused warnings in tests/* Beautify main.rs and lib.rs WIP: Move tests Move src/test/harness to tests/support/harness Split RLS into bin/lib Update Clippy Change all mentions of `rls-preview` to `rls` Make config mutex borrow scope explicit Fallback to racer definition ```` Fixes rls build.
Changes: ```` Rustfmt all the things Don't make decisions on values that don't represent the decision Rustup Actually check for constants. formatting fix Update clippy_lints/src/needless_bool.rs formatting fix needless bool lint suggestion is wrapped in brackets if it is an "else" clause of an "if-else" statement Remove negative integer literal checks. Fix `implicit_return` false positives. ````
|
pulled in a clippy update as well.... |
|
📌 Commit 7f8e4aa has been approved by |
submodules: update rls and clippy Changes: ```` Remove state.analysis due to Rust PR #57476 Improve missing nightly readme info Bump languageserver-types to v0.54.0 and renam crate name to lsp-types Delete bors.toml Fix tests Fix rust-lang/rls#1231 Implement asynchronous message reading Use typed requests Implement Tokio-based test LSP client Update README.md to account for Travis url change Simplify wait_for_all recv calls Update dependencies Revert NLL bug workaround Remove old test_data entry in .gitignore Reorganize some tests Don't test RLS binary target directly Move tooltip tests to integration tests Simplify tooltip test harness Only use FIXTURES_DIR to determine fixtures Remove src/test/mod.rs Centralise FIXTURES_DIR across unit and integration tests Move lens test to tests/ Suppress unused warnings in tests/* Beautify main.rs and lib.rs WIP: Move tests Move src/test/harness to tests/support/harness Split RLS into bin/lib Update Clippy Change all mentions of `rls-preview` to `rls` Make config mutex borrow scope explicit Fallback to racer definition ```` Fixes rls build.
|
☀️ Test successful - checks-travis, status-appveyor |
|
📣 Toolstate changed by #57805! Tested on commit 4c2be9c. 🎉 clippy-driver on windows: build-fail → test-pass (cc @Manishearth @llogiq @mcarton @oli-obk, @rust-lang/infra). |
Tested on commit rust-lang/rust@4c2be9c. Direct link to PR: <rust-lang/rust#57805> 🎉 clippy-driver on windows: build-fail → test-pass (cc @Manishearth @llogiq @mcarton @oli-obk, @rust-lang/infra). 🎉 clippy-driver on linux: build-fail → test-pass (cc @Manishearth @llogiq @mcarton @oli-obk, @rust-lang/infra). 🎉 rls on windows: build-fail → test-pass (cc @nrc @Xanewok, @rust-lang/infra). 🎉 rls on linux: build-fail → test-pass (cc @nrc @Xanewok, @rust-lang/infra).
Changes:
Fixes rls build.