This patch adds bare-bone VS Code integration for development purposes of clangd.
Details
Diff Detail
- Build Status
Buildable 4087 Build 4087: arc lint + arc unit
Event Timeline
| clangd/clients/clangd-vscode/src/extension.ts | ||
|---|---|---|
| 1 | ES6 modules are always in strict mode, you can drop this here. | |
| 7 | use /** JSDoc comments */ so that you get these comments on hover in the IDE. | |
| 11 | prefer to always use const for variables that don't get re-assigned. | |
| 17 | .h? | |
| 23 | remove the tutorial comments? | |
| 28 | this comment does not seem to fit the next line | |
| clangd/clients/clangd-vscode/test/extension.test.ts | ||
| 2 | remove the tutorial comments? | |
| clangd/clients/clangd-vscode/tsconfig.json | ||
| 7 | As this runs in a recent Node with pretty complete ES6 support, consider including:
These add declarations for a bunch of useful library builtins, e.g. for array iteration, String.repeat, and so on. | |
| 10 | tsc supports a bunch of strictness options that you should enable. See here for the reference: https://www.typescriptlang.org/docs/handbook/compiler-options.html Enable:
The following are a bit questionable (I prefer to use tslint for them as they break the workflow):
| |
| clangd/clients/clangd-vscode/vsc-extension-quickstart.md | ||
| 1 | drop or update? | |
ES6 modules are always in strict mode, you can drop this here.