l10n sample README feedback #162023
Labels
author-verification-requested
Issues potentially verifiable by issue author
bug
Issue identified by VS Code Team member as probable bug
l10n-platform
Localization platform issues (not wrong translations)
verified
Verification succeeded
Milestone
Testing #161870
Small typo in https://github.com/microsoft/vscode-extension-samples/blob/990c4253726858f4457e73bd279faf1c713949d3/l10n-sample/README.md#L24 -- either missing a second option to run an extension using a proposed API (in extension development host), or the 'either' should be removed?
For
https://github.com/microsoft/vscode-extension-samples/blob/990c4253726858f4457e73bd279faf1c713949d3/l10n-sample/README.md#L52-L58
I wonder if we can update https://github.com/microsoft/vscode-generator-code to scaffold the l10n directory, add the required "l10n" key to the package.json (once the proposed API is finalized of course), and even include a sample script like the cli.ts. Then we could also have a step in the extension generator which asks you "Do you want to set up localization for this extension" or similar. Also, is it common that extensions will have to use @vscode/l10n (my understanding is it's for extensiosn which have strings that need localizing in subprocesses), and if not, could we move that section below @vscode/l10n-dev?
Minor nit, "it's" -> "its" on https://github.com/microsoft/vscode-extension-samples/blob/990c4253726858f4457e73bd279faf1c713949d3/l10n-sample/README.md#L62
The text was updated successfully, but these errors were encountered: