[PATCH] D68807: [ClangTidy] Separate tests for infrastructure and checkers

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 15:21:32 PDT 2019


gribozavr added a comment.

It would look weird if we have a ton of tests for checkers in a directory, and then a subdirectory for infra tests.

Why are we trying to optimize the number of renamed files? I already did the renaming, so we are not saving any work. I think we should optimize for long-term readability of the codebase.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68807





More information about the cfe-commits mailing list