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

[Marina Kalashina via Phabricator via cfe-commits]

MarinaKalashina added a comment.

@alexfh Just for me to be sure, should there be the following structure in http://clang.llvm.org/extra/index.html:
Clang-Tidy

  The list of clang-tidy checks
  Using clang-tidy
  Suppressing Undesired Diagnostics
  Clang-tidy Integrated (separate page)
  Getting Involved (separate page)

And should the latter two pages be linked from 'Clang-Tidy' index.rs like 'The list of clang-tidy checks', which is placed in 'See also'?
Thank you.



================
Comment at: docs/clang-tidy/index.rst:262
+
+Clang-tidy integrated
+---------------------
----------------
alexfh wrote:
> So how about moving this chapter to a separate page and adding a link to it at the top of this document? While certainly useful, the information in this section has a different focus from the rest of the documentation. Given that the document has grown significantly, I suggest to split out this section and the "Getting involved" section. (The latter can be done separately)
Just for me to be sure, should there be the following structure in http://clang.llvm.org/extra/index.html:
Clang-Tidy
  The list of clang-tidy checks
  Using clang-tidy
  Suppressing Undesired Diagnostics
  Clang-tidy Integrated (separate page)
  Getting Involved (separate page)
And should the latter two pages be linked from 'Clang-Tidy' index.rs like 'The list of clang-tidy checks', which is placed in 'See also'?
Thank you.



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

https://reviews.llvm.org/D54945