[PATCH] D118104: Make run-clang-tidy.py print the configured checks correctly

Jesko Appelfeller via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 3 00:48:35 PST 2022


JesApp added a comment.

Well, since this was more of source of confusion than actual incorrect behaviour, I don't think there should be a test for it.

In general though, I think the script is complex enough to warrant some testing. That being said: I don't think they should be part of this patch. Also, I'm doing this on company time and I don't think my boss would be to happy if I wrote a testsuite from scratch when I just wanted to fix one bug. :D


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

https://reviews.llvm.org/D118104



More information about the cfe-commits mailing list