This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Improve documentation on Clangd integration
ClosedPublic

Authored by kbobyrev on Sep 15 2020, 4:50 AM.

Details

Summary

The integration is already complete; this patch updates information as well as
suggests using Clang-Tidy via Clangd integration that is vastly available
in most editors through LSP client plugins.

Diff Detail

Event Timeline

kbobyrev created this revision.Sep 15 2020, 4:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 15 2020, 4:50 AM
kbobyrev requested review of this revision.Sep 15 2020, 4:51 AM
hokein added inline comments.Sep 15 2020, 7:06 AM
clang-tools-extra/docs/clang-tidy/Integrations.rst
5–7

nit: update to the official website http://clangd.llvm.org/

22

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 +.

Thanks for doing this! (Agree with Haojian's comments too)

clang-tools-extra/docs/clang-tidy/Integrations.rst
10–15

nit: lowercase C for clangd

kbobyrev updated this revision to Diff 292149.Sep 16 2020, 2:06 AM
kbobyrev marked 3 inline comments as done.

Address review comments.

hokein accepted this revision.Sep 16 2020, 3:06 AM

thanks!

This revision is now accepted and ready to land.Sep 16 2020, 3:06 AM
This revision was landed with ongoing or failed builds.Sep 16 2020, 3:10 AM
This revision was automatically updated to reflect the committed changes.