DmitryPolukhin added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:320-324 + if (!Checks.empty()) { + llvm::errs() << "Error: --config-file and --checks are mutually " + "exclusive. Specify only one.\n"; + return nullptr; + } ---------------- DmitryPolukhin wrote: > njames93 wrote: > > I disagree with this check here, `Config` is not mutually exclusive with > > `Checks`, `Checks` gets applied atop of `Config`. So the same should happen > > when using `ConfigFile` with `Checks` > +1 Clarify: +1 to @njames93 that we don't need this check. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89936/new/ https://reviews.llvm.org/D89936 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits