[PATCH] D60115: Adding 'CLion integration' to clang-format doc

MarinaKalashina added a comment. @sylvestre.ledru Thanks a lot for your help Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60115/new/ https://reviews.llvm.org/D60115 ___ cfe-commits mailing list cfe-commits@lists.llv

[PATCH] D60115: Adding 'CLion integration' to clang-format doc

MarinaKalashina marked an inline comment as done. MarinaKalashina added a comment. @sylvestre.ledru Thank you! Could you please explain how I should proceed with the accepted revision now (without commit access, do I need to ask someone to make the commit for me)? CHANGES SINCE LAST ACTION h

[PATCH] D60115: Adding 'CLion integration' to clang-format doc

MarinaKalashina updated this revision to Diff 193441. MarinaKalashina added a comment. added a link to CLion CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60115/new/ https://reviews.llvm.org/D60115 Files: docs/ClangFormat.rst Index: docs/ClangFormat.rst

[PATCH] D60115: Adding 'CLion integration' to clang-format doc

MarinaKalashina created this revision. MarinaKalashina added reviewers: djasper, sylvestre.ledru, alexfh. Herald added subscribers: cfe-commits, jdoerfert. Herald added a project: clang. This commit adds a chapter 'CLion integration' to ClangFormat.rst. The official announcement of clang-format s

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

MarinaKalashina added a comment. @Eugene.Zelenko @alexfh Thank you! How should I proceed with the revision now? Do I get it right from https://llvm.org/docs/Phabricator.html#committing-a-change, that without commit access, I need to ask for committing the changes for me? CHANGES SINCE LAST A

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

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 script > warnings. The script warns abou

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

MarinaKalashina updated this revision to Diff 181528. MarinaKalashina marked an inline comment as done. MarinaKalashina added a comment. files renamed: //contribution.rst// to //Contributing.rst//, // integrations.rst// to //Integrations.rst// CHANGES SINCE LAST ACTION https://reviews.llvm.or

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

MarinaKalashina updated this revision to Diff 181217. MarinaKalashina added a comment. restoring full content CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 Files: docs/clang-tidy/contribution.rst docs/clang-tidy/index.rst docs/clang-tidy

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

MarinaKalashina updated this revision to Diff 181216. MarinaKalashina added a comment. integrations.rst - end of file new line CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 Files: docs/clang-tidy/integrations.rst Index: docs/clang-tidy/int

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

MarinaKalashina added a comment. @alexfh Thank you, please see the updated structure. @Eugene.Zelenko Done, the only warnings I got were about the table rows width. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 __

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

MarinaKalashina updated this revision to Diff 181018. MarinaKalashina added a comment. Extracting 'Getting Involved' and 'Clang-tidy integrations' to separate pages linked in See Also. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 Files: do

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

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

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

MarinaKalashina added a comment. @ilya-biryukov Thank you for a quick response and sorry for so many iterations. I uploaded a full-context version; hopefully this time it's alright - on my side, it finally looks like a proper full-context diff. CHANGES SINCE LAST ACTION https://reviews.llvm

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

MarinaKalashina updated this revision to Diff 178647. MarinaKalashina added a comment. Full-context patch CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 Files: docs/clang-tidy/index.rst Index: docs/clang-tidy/index.rst ==

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

MarinaKalashina updated this revision to Diff 178623. MarinaKalashina set the repository for this revision to rCTE Clang Tools Extra. MarinaKalashina added a comment. Another attempt to update the patch with the following fixes: - empty line before 'Standalone tool' - table columns with '+/-' ali

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

MarinaKalashina added a comment. @alexfh Thanks a lot for your patience and help. I've made another revision, now with the diff made by 'git show HEAD -U99' to have the full context availlable. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D5494

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

MarinaKalashina updated this revision to Diff 176601. MarinaKalashina added a comment. Full-context patch for the changes introduced in the previous commit. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54945/new/ https://reviews.llvm.org/D54945 Files: docs/clang-tidy/index.rst Ind

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

MarinaKalashina marked 4 inline comments as done. MarinaKalashina added a comment. In D54945#1318278 , @alexfh wrote: > In D54945#1316251 , @MarinaKalashina > wrote: > > > Fixes: > > > > - empty line before 'Standa

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

MarinaKalashina updated this revision to Diff 176328. MarinaKalashina added a comment. 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'

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

MarinaKalashina created this revision. MarinaKalashina added a reviewer: asl. Herald added subscribers: cfe-commits, arphaman. Proposing changes to //clang-tools-extra/docs/clang-tidy/index.rst// 'Using clang-tidy' was split into 2 chapters: - 'Standalone tool' (the previous content of 'Using cl