[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Thanks for the context! In D75985#1917163 , @sylvestre.ledru wrote: > With my Debian/Ubuntu hat, i have been interested to write some dumb > integration tests to be sure that clangd, as a package, works as expected. > I just

[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. Good question :) I have been wearing two hats here. With my Debian/Ubuntu hat, i have been interested to write some dumb integration tests to be sure that clangd, as a package, works as expected. I just need to make sure that the binary starts, that I can send s

[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru updated this revision to Diff 249653. sylvestre.ledru added a comment. fix some typo + desc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75985/new/ https://reviews.llvm.org/D75985 Files: clang-tools-extra/docs/clangd/SimpleExamp

[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Thanks Sylvestre! We're actually in the process of moving docs to a new site (preview: https://clangd.github.io - once set up it will be at clangd.llvm.org). Who do you think is the audience/purpose for this doc? End-users don't need to understand this to use clangd

[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru updated this revision to Diff 249595. sylvestre.ledru added a comment. Add a link to the doc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75985/new/ https://reviews.llvm.org/D75985 Files: clang-tools-extra/docs/clangd/SimpleExam

[PATCH] D75985: clangd doc: Show a test case for clangd with some commands

2020-03-11 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru created this revision. sylvestre.ledru added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. sylvestre.ledru updated this revision to Diff 249595. sylvestre.ledru added a co