njames93 added a comment.

In D93459#2460445 <https://reviews.llvm.org/D93459#2460445>, @fodinabor wrote:

> Btw. is there a way to add tests for command-line options, so this won't 
> happen unnoticed again?

I'd guess some lit tests could be used to verify the command line syntax is 
correct

  // RUN: clang-format --Wno-error=unknown --style=`{UnknownKey: true}` 
--dump-config

If it can't parse that command line, or it maybe parses it incorrectly, 
clang-format will return non-zero and fail the test


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93459

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

Reply via email to