[PATCH] D147955: [clang-tidy] Extend CheckOptions to support grouping checks options

[Nathan James via Phabricator via cfe-commits]

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