[PATCH] D147876: [clang-tidy] Support specifying checks as a list in the config file

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 10 23:06:31 PDT 2023


njames93 added a comment.

In D147876#4257197 <https://reviews.llvm.org/D147876#4257197>, @carlosgalvezp wrote:

> In D147876#4256182 <https://reviews.llvm.org/D147876#4256182>, @njames93 wrote:
>
>> with checked in configuration files
>
> I'm not sure I understand what you mean by "checked in", could you clarify?

When the `.clang-tidy` file is checked into the source control(git). It means that anyone who contributes to the project will need to ensure that they have a version of clang-tidy that will be able to read the file.
This can cause problems as binaries of clang-tidy aren't always provided for specific versions on certain platforms.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147876



More information about the cfe-commits mailing list