i18n: localize Cmd command output messages (15 MessageIds)#2940
i18n: localize Cmd command output messages (15 MessageIds)#2940gordonlu wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
There was a problem hiding this comment.
Code Review
This pull request localizes command-line output messages across several commands, including /trust, /lsp, /logout, /queue, and /task. It introduces a new error_localized helper to format localized error messages with a localized prefix, adds new message IDs to the localization system, and provides translations for English, Vietnamese, Chinese, Japanese, Portuguese, and Spanish. There are no review comments, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Thanks @gordonlu — we merged #2891 and #2896 from your i18n batch tonight, which moved Since the slices all touch |
Localize all Cmd* command output messages across 7 shipped locales.
What's covered
CommandResult::error_localized()Changes
localization.rs: 15 new MessageIds + translations in all 7 localescommands/mod.rs: newCommandResult::error_localized()with localized error prefixcommands/task.rs: all usage error messages viatr()commands/config.rs: trust/lsp/logout messages viatr()commands/queue.rs: editing-queued-message header viatr()Testing
-D warningscargo fmt --all --checkclean