Help to fix https://github.com/clangd/clangd/issues/159.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 38590 Build 38589: arc lint + arc unit
Event Timeline
clang-tools-extra/clangd/clients/clangd-vscode/package.json | ||
---|---|---|
38 | Just to make sure I understand what's going on... Could you explain what --baseImagesUrl does? |
clang-tools-extra/clangd/clients/clangd-vscode/package.json | ||
---|---|---|
38 | This is expected, this is a base url. it is used to assemble a full url with the screenshot links in the readme, an example is https://raw.githubusercontent.com/llvm/llvm-project/master/clang-tools-extra/clangd/clients/clangd-vscode/ + doc-assets/complete.png, so that the screenshot can be rendered correctly in the VSCode market. |
LGTM
clang-tools-extra/clangd/clients/clangd-vscode/package.json | ||
---|---|---|
38 | Thanks for the clarification! Surprised we need this for both package and publish, I'd expect this to be handled in either first or the second step, but not both. I guess I don't fully understand what they do with this URL. NIT: could add a comment mentioning that this URL is used to "host images (e.g. screenshots in documentation)"? |
clang-tools-extra/clangd/clients/clangd-vscode/package.json | ||
---|---|---|
38 |
Actually, when we make a new release, we use publish which includes the package. The package is usually used to build a local package (mainly for debugging purpose).
Unfortunately, comment is not permitted in the JSON :( |
Just to make sure I understand what's going on...
Could you explain what --baseImagesUrl does?
The suggested url returns an error code when I try to load it in a browser.