[PATCH] D147955: [clang-tidy] Extend CheckOptions to support grouping checks options
Nathan James via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 10 15:54:47 PDT 2023
njames93 added a comment.
In D147955#4256649 <https://reviews.llvm.org/D147955#4256649>, @PiotrZSL wrote:
> There are so many ways of defining check configurations, and I never heard of them.
> Maybe some separate section in documentation is needed for them.
The current documentation only defines this method
CheckOptions:
<check_name.option_name>: <option_value>
It doesn't mention flow syntax `{CheckOptions: {<check_name.option_name>: <option_value>, ...}}` because thats implicit when using yaml.
The other syntax of using
CheckOptions:
- key: <check_name.option_name>
value: <option_value>
- key: ...
value: ...
And its Flow counterpart `{CheckOptions: [{key: <check_option.option_name>, value: <option_value>}, {...} ]}` were removed from the documentation as an old deprecated format, left for compatibility with old config files
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147955/new/
https://reviews.llvm.org/D147955
More information about the cfe-commits
mailing list