[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 12:51:52 PDT 2023


njames93 added a comment.

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

> Thanks for the review @njames93 ! Do you think it makes sense that we deprecate the string format, so that we only support the list format? To be fully removed in clang-tidy 19. Only for config file, for `--checks` we can still support string. This would allow us to get rid of all the code I wrote and simply pass the vector to `IO.mapOptional` :)

Deprecating options from the config is a right pain to do and discourages upgrading clang-tidy version with checked in configuration files


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