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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to