[PATCH] D111870: [clangd] Add a way to enable IncludeCleaner through config

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev marked 9 inline comments as done.
kbobyrev added inline comments.


================
Comment at: clang-tools-extra/clangd/IncludeCleaner.cpp:215
+    D.Message = "Included header is unused";
+    D.Name = "clangd-include-cleaner";
+    // FIXME: This range should be the whole line with target #include.
----------------
sammccall wrote:
> sammccall wrote:
> > Source should be `Clangd` I think (our first?).
> > 
> > Name should be something like "unused-include": this doesn't need to echo the tool name.
> clangd-unused-header -> unused-header
> 
> (there's a test, but it's stale)
what do you mean by "stale"?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111870/new/

https://reviews.llvm.org/D111870