[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