[libcxx-commits] [PATCH] D117174: [libc++][test] Run clang-tidy during CI

Thu Feb 17 06:27:17 PST 2022

ldionne added a comment.

In D117174#3326677 <https://reviews.llvm.org/D117174#3326677>, @philnik wrote:

> I'll look into using the just built clang-tidy and checking the clang-tidy version, but I don't think it's worth reverting this patch for that. If anybody thinks it should be reverted LMK.

@philnik I think what we should do is check in `features.py` that the clang-tidy version is sufficiently recent. We shouldn't try to use the just-built clang-tidy unless it doesn't add many dependencies to the build (but I doubt that).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117174