[PATCH] D140462: [clangd] Add schema for `.clangd` config
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 23 01:18:40 PST 2022
sammccall added a comment.
I think there's a clear benefit, but my feeling is it's outweighed by the costs, which are fairly high given lack of any automation/tests.
If clangd contributors should maintain this, how should contributors/reviewers know whether changes are correct?
If they needn't maintain it, then having it on a third-party site seems appropriate. Maybe a `/contrib/` directory is a useful compromise?
This is definitely informed by my biases: I have to make/review changes that touch config, don't have a good understanding of JSON-schema, and I don't use VSCode.
---
One practical question: AIUI the main point of having this is so it can be provided to the VSCode YAML extension so it understands .clangd files. How does this work mechanically? Does it need to be part of the vscode-clangd repo instead?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140462/new/
https://reviews.llvm.org/D140462
More information about the cfe-commits
mailing list