[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