[PATCH] D60560: [clangd] Enable clang-tidy by default.

2019-04-12 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL358282: [clangd] Enable clang-tidy by default. (authored by hokein, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.o

[PATCH] D60560: [clangd] Enable clang-tidy by default.

2019-04-12 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. Yay! LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60560/new/ https://reviews.llvm.org/D60560

[PATCH] D60560: [clangd] Enable clang-tidy by default.

2019-04-11 Thread Haojian Wu via Phabricator via cfe-commits
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