SAtacker marked an inline comment as done.
SAtacker added a comment.

Gentle ping @alexfh @JonasToth 
Thank you.



================
Comment at: clang-tools-extra/clang-tidy/tool/run-clang-tidy.py:228
                       'default')
-  parser.add_argument('-config', default=None,
+  group = parser.add_mutually_exclusive_group()
+  group.add_argument('-config', default=None,
----------------
JonasToth wrote:
> nit: i think `config_group` or so would be better to show that its only 
> config-related. This name is too generic i feel.
Thanks, done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120331

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to