[PATCH] D52334: [clang-tidy] Build it even without static analyzer

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 29 10:46:47 PDT 2018


steveire marked 2 inline comments as done.
steveire added a comment.

> I think a reasonable place would be docs/clang-tidy/index.rst in the "Getting Involved" area.

That's at least actionable, but not very specific. I've added docs now. If they don't say what you want them to say, then please be specific about what you want them to say. There's no reason to have 5 more rounds of review here if being specific means only one more round is needed.

Are you going to 'accept' this patch eventually, or will I still be waiting for @alexfh once you're happy with it?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52334





More information about the cfe-commits mailing list