[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

This revision was automatically updated to reflect the committed changes. Closed by commit rGfbf611ed2a76: [clang-tidy] Extend spelling for CheckOptions (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128337/new/ https://review

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

njames93 updated this revision to Diff 439478. njames93 added a comment. Rebase to check CI Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128337/new/ https://reviews.llvm.org/D128337 Files: clang-tools-extra/clang-tidy/ClangTidyOptions.cpp cla

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D128337#3602784 , @njames93 wrote: > In D128337#3602694 , @aaron.ballman > wrote: > >> I li

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

njames93 updated this revision to Diff 439139. njames93 marked an inline comment as done. njames93 added a comment. Rebased and addressed comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128337/new/ https://reviews.llvm.org/D128337 Files:

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

njames93 added a comment. In D128337#3602694 , @aaron.ballman wrote: > I like the changes -- this is a much nicer syntax for specifying > configuration options! > >> The only observable differences are support for the new syntax and >> -dump=config wil

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

aaron.ballman added a comment. I like the changes -- this is a much nicer syntax for specifying configuration options! > The only observable differences are support for the new syntax and > -dump=config will emit using the new syntax. Do you expect the behavior of `-dump` to cause any problems

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

njames93 updated this revision to Diff 439018. njames93 added a comment. Herald added subscribers: usaxena95, kadircet. Update some test files using new syntax. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128337/new/ https://reviews.llvm.org/D128

[PATCH] D128337: [clang-tidy] Extend spelling for CheckOptions

njames93 created this revision. njames93 added reviewers: aaron.ballman, alexfh. Herald added subscribers: carlosgalvezp, arphaman, xazax.hun. Herald added a project: All. njames93 requested review of this revision. Herald added subscribers: cfe-commits, aheejin. Herald added a project: clang-tools