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

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Feb 4 14:31:54 PST 2022


philnik added a comment.

In D117174#3298131 <https://reviews.llvm.org/D117174#3298131>, @ldionne wrote:

> If `clang-tidy` does provide coverage for all that `libcxx/test/libcxx/lint/lint_headers.sh.py` does (and I think it does), I would remove it too. Maintaining less infrastructure/testing code is better in the long term.
>
> @philnik Can you do that in this patch?
>
> We will have to wait for the nodes to get the updated image before this can be checked-in... for some reason this is terribly long..

I'll update the patch tomorrow. I don't want to block CI for other patches right now. Is there any way to check if the nodes have clang-tidy installed without re-running CI?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117174



More information about the libcxx-commits mailing list