[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