feat: support multi-node batch installation in Skills Hub#12921
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 085d20cf17
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| AppInstallTask: "应用安装任务" | ||
| PushAppFailed: "推送应用安装任务失败" | ||
| SkillHubInstallTask: "安装技能 [{{ .name }}]" | ||
| SkillHubInstallToTarget: "安装技能 {{ .name }} 到目标 {{ .target }}" |
There was a problem hiding this comment.
Use the same placeholder for target installs
This locale is the only new SkillHubInstallToTarget message that requires both .name and .target; the other locales only require .name, and the i18n helpers commonly localize messages with a single {name: ...} value. In the Chinese locale, that same target-install task title would render the missing target as <no value> instead of showing the target name, so keep the placeholder contract consistent with the other translations or update the caller to always provide target.
Useful? React with 👍 / 👎.
No description provided.