DmitryPolukhin added a comment.
In D75184#2093089 <https://reviews.llvm.org/D75184#2093089>, @njames93 wrote:
> As an afterthought, Would you think it a good idea to extend this logic to
> the `ConfigOptionsProvider` That way you can use something along the lines of:
>
> clang-tidy --config='{InheritParentConfig: true, CheckOptions: [<Options
> overriding/complimenting .clang-tidy files>]}'
>
>
> clang-tidy would then go to read the .clang-tidy configuration files in much
> the same was as is done here. This would let you set custom configuration
> options on the command line on top of whats in configuration files.
> Obviously is `InheritParentConfig` is unset or `false` then we wouldn't go
> looking for .clang-tidy files.
It sounds logical. I would even extend it a little bit to allow case
`--config=@filepath` where filepath file will be read and expanded as the
highest priority config. @njames93 are you going to work on this?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75184/new/
https://reviews.llvm.org/D75184
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
