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

Marina Kalashina via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 4 03:57:04 PST 2018


MarinaKalashina marked 4 inline comments as done.
MarinaKalashina added a comment.

In D54945#1318278 <https://reviews.llvm.org/D54945#1318278>, @alexfh wrote:

> In D54945#1316251 <https://reviews.llvm.org/D54945#1316251>, @MarinaKalashina wrote:
>
> > Fixes:
> >
> > - empty line before 'Standalone tool'
> > - table columns with '+/-' aligned
> > - line width limited to 80 (except for the table)
> >
> >   Additions:
> > - clang-tidy-vs plugin
> > - Clangd in the intro, the table, and CLion's paragraph
>
>
> Did you forget to add a new file to the patch?
>
> Please also include full context into the diff. See https://llvm.org/docs/Phabricator.html


Sorry but the new diff was included to the patch.. I can see the updates as Diff 176328. Could you please check and let me know if it does not work? Thank you.


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

https://reviews.llvm.org/D54945





More information about the cfe-commits mailing list