[PATCH] D54945: This commit adds a chapter about clang-tidy integrations

Marina Kalashina via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 14 04:24:27 PST 2019


MarinaKalashina added a comment.

@Eugene.Zelenko

> I would suggest to rename contribution to Contributing (see LLVM documentation) and integrations to Integrations.

Sure, done.

> Will be also good idea to fix D55523 <https://reviews.llvm.org/D55523> script warnings.

The script warns about double spaces and line width in the table, for example:

  warning: line 39 contains double spaces
  warning: line 39 is in excess of 80 characters (196)

These spaces and width are intended since they 'draw' the table layout. Do you think we could ignore such warnings in this particular case?


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

https://reviews.llvm.org/D54945





More information about the cfe-commits mailing list