[PATCH] D87686: [clang-tidy] Improve documentation on Clangd integration

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 15 07:06:48 PDT 2020


hokein added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/Integrations.rst:5
 
 .. _Clangd: https://clang.llvm.org/extra/clangd.html
+.. _is available: https://clangd.llvm.org/installation.html#editor-plugins
----------------
nit: update to the official website http://clangd.llvm.org/


================
Comment at: clang-tools-extra/docs/clang-tidy/Integrations.rst:22
 +--------------------------------------+------------------------+---------------------------------+--------------------------+-----------------------------------------+--------------------------+
+|Clangd                                |         \+\            |               \-\               |           \-\            |                 \-\                     |           \-\            |
++--------------------------------------+------------------------+---------------------------------+--------------------------+-----------------------------------------+--------------------------+
----------------
hmm, I know the purpose, but the table is sorted by the name, I think we should not change it.

btw `.clang-tidy` is supported now, we can change `-` to `+`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87686



More information about the cfe-commits mailing list