[clang-tools-extra] [clang-tidy] Avoid processing declarations in system headers (PR #128150)

[Carlos Galvez via cfe-commits]

Carlos =?utf-8?q?Gálvez?= <carlos.galvez at zenseact.com>,
Carlos =?utf-8?q?Gálvez?= <carlos.galvez at zenseact.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/128150 at github.com>


carlosgalvezp wrote:

Thank you for the discussion, I'm okey with changing the code based on the feedback, namely: 

- move the logic to ASTMatchers, off by default
- clang tidy actively enables it
- document technical debt with FIXME/GH ticket

> Properly update the documentation and changelog

I did that in the release notes. It's there anything else you believe is missing? I will update the AST documentation accordingly as well.

https://github.com/llvm/llvm-project/pull/128150