[PATCH] D79095: [clang-format] NFC Correct clang-format headers file so documentation can be once again autogenerated

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 30 01:49:08 PDT 2020


krasimir added a comment.

In D79095#2010271 <https://reviews.llvm.org/D79095#2010271>, @MyDeveloperDay wrote:

> In D79095#2010250 <https://reviews.llvm.org/D79095#2010250>, @krasimir wrote:
>
> > Awesome! Thank you!
> >  Now that we've got "clang/phabricator/lint/git" robots that leave comments on patches, we may want to think about a way to have this automatically enforced in the future.
>
>
> Thats a great idea.. do you know who looks after those scripts?


I asked around and a simple thing we can do that would have the effect of complaining on the Phabricator patch if the format options doc has to be regenerated is a lit test that does a diff.
Will prepare a patch, after this lands, so that the newly added test doesn't fail at first.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79095





More information about the cfe-commits mailing list