[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

carlosgalvezp added a comment. No worries, it's indeed good practice! Feel free to take a look at the docs, where this (and many more features) is described :) https://clang.llvm.org/extra/clang-tidy/ > will disable all default checks (-*) and enable all clang-analyzer-* checks > except for cl

[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

reyg abandoned this revision. reyg added a comment. Well now I feel stupid for not realizing that, thanks for pointing it out :) At least I learned how to hack around in clang-tidy 😄 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116163/new/ https:

[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

carlosgalvezp added a comment. In D116163#3206473 , @reyg wrote: > I actually couldn't find a way to black-list specific checks. Is there a way > to do that? Just prepend a dash to the check name that you want to disable: `checks=cppcoreguidelines-*,-c

[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

reyg added a comment. I actually couldn't find a way to black-list specific checks. Is there a way to do that? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116163/new/ https://reviews.llvm.org/D116163

[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

carlosgalvezp added a comment. Could you summarize what problem this is solving? There's multiple ways to "ignore diagnostics from specific checks", I think it would be confusing to add yet another one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D116163: [clang-tidy] Add a SuppressedChecks option to ignore diagnostics from specific checks

reyg created this revision. reyg added a reviewer: cfe-commits. Herald added subscribers: carlosgalvezp, xazax.hun. reyg requested review of this revision. Herald added a project: clang-tools-extra. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D116163 Files: clang-tools-extra