[PATCH] D141000: [clang-tidy] Introduce HeaderFileExtensions option

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 4 10:54:38 PST 2023


carlosgalvezp added a comment.

In D141000#4026600 <https://reviews.llvm.org/D141000#4026600>, @Eugene.Zelenko wrote:

> Looks OK for me, but probably will be good idea to remove check-specific options in same commit.

Thanks for the review! Sure, I can do that, I just thought we should try to keep commits as minimal as possible. I also wonder if we need to follow a formal deprecation process or we can just remove the option from existing checks directly?

@njames93 , as Code Owner, could you give your view on the general direction of the patch before I move forward? Thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141000/new/

https://reviews.llvm.org/D141000



More information about the cfe-commits mailing list