[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