[clang-tools-extra] r358282 - [clangd] Enable clang-tidy by default.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 12 06:07:48 PDT 2019


Author: hokein
Date: Fri Apr 12 06:07:48 2019
New Revision: 358282

URL: http://llvm.org/viewvc/llvm-project?rev=358282&view=rev
Log:
[clangd] Enable clang-tidy by default.

Summary:
We have turned on the flag internally for a while, and we don't receive complains.
Should be good to turn it on now.

If the projects doesn't have .clang-tidy files, no clang-tidy check will
be run.

Reviewers: sammccall

Subscribers: ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D60560

Modified:
    clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp

Modified: clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp?rev=358282&r1=358281&r2=358282&view=diff
==============================================================================
--- clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp (original)
+++ clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp Fri Apr 12 06:07:48 2019
@@ -228,7 +228,7 @@ static llvm::cl::opt<std::string> ClangT
 static llvm::cl::opt<bool> EnableClangTidy(
     "clang-tidy",
     llvm::cl::desc("Enable clang-tidy diagnostics."),
-    llvm::cl::init(false));
+    llvm::cl::init(true));
 
 static llvm::cl::opt<bool> SuggestMissingIncludes(
     "suggest-missing-includes",




More information about the cfe-commits mailing list