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



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

Reply via email to