[PATCH] D68080: [clangd][vscode] Add npm helper commands to package/release the extension.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 26 07:01:37 PDT 2019
hokein marked an inline comment as done.
hokein added inline comments.
================
Comment at: clang-tools-extra/clangd/clients/clangd-vscode/package.json:38
+ "package": "vsce package --baseImagesUrl https://raw.githubusercontent.com/llvm/llvm-project/master/clang-tools-extra/clangd/clients/clangd-vscode/",
+ "publish": "vsce publish --baseImagesUrl https://raw.githubusercontent.com/llvm/llvm-project/master/clang-tools-extra/clangd/clients/clangd-vscode/"
},
----------------
ilya-biryukov wrote:
> hokein wrote:
> > ilya-biryukov wrote:
> > > 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.
> > 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.
> 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)"?
>
>
> 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.
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).
> NIT: could add a comment mentioning that this URL is used to "host images (e.g. screenshots in documentation)"?
Unfortunately, comment is not permitted in the JSON :(
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68080/new/
https://reviews.llvm.org/D68080
More information about the cfe-commits
mailing list