hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. Herald added a project: clang.
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. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D60560 Files: clang-tools-extra/clangd/tool/ClangdMain.cpp Index: clang-tools-extra/clangd/tool/ClangdMain.cpp =================================================================== --- clang-tools-extra/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -228,7 +228,7 @@ 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",
Index: clang-tools-extra/clangd/tool/ClangdMain.cpp =================================================================== --- clang-tools-extra/clangd/tool/ClangdMain.cpp +++ clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -228,7 +228,7 @@ 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",
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits