10569: fix: Skip non clippy completions when completing a clippy path r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
This commit is contained in:
bors[bot] 2021-10-17 21:05:10 +00:00 committed by GitHub
commit 91cbda43c2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -50,8 +50,13 @@ pub(super) fn complete_lint(
if lint_already_annotated { if lint_already_annotated {
continue; continue;
} }
let insert = match qual { let insert = match (qual, ctx.previous_token_is(T![:])) {
Some(qual) if !ctx.previous_token_is(T![:]) => format!("{}::{}", qual, name), (Some(qual), false) => format!("{}::{}", qual, name),
// user is completing a qualified path but this completion has no qualifier
// so discard this completion
// FIXME: This is currently very hacky and will propose odd completions if
// we add more qualified (tool) completions other than clippy
(None, true) => continue,
_ => name.to_owned(), _ => name.to_owned(),
}; };
let mut item = let mut item =