bito-code-review[bot] commented on code in PR #41649: URL: https://github.com/apache/superset/pull/41649#discussion_r3510503612
########## superset/translations/ru/LC_MESSAGES/messages.po: ########## @@ -996,11 +1014,17 @@ msgstr "Роль успешно создана." msgid "API key name is required" msgstr "Требуется имя таблицы" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Incorrect translation — API key context</b></div> <div id="fix"> The msgstr for `API key name is required` (line 1015) reads "Требуется имя таблицы" (table name required), which is semantically incorrect. Adjacent entries in the same file confirm this is an API key context: `API Key Created` (line 1002), `API Keys` (line 1006), `API key created successfully` (line 1010), `API key revoked successfully` (line 1020). The translation should reflect "name" of an "API key", not a database table. </div> </div> <small><i>Code Review Run #c5b8fe</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them ########## superset/translations/ru/LC_MESSAGES/messages.po: ########## @@ -2274,13 +2304,19 @@ msgstr "Автоматически" msgid "Auto Zoom" msgstr "Автомасштабирование" -#, python-format +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr, +# sr_Latn] +#, fuzzy, python-format msgid "Auto refresh paused (set to %s seconds)" -msgstr "" +msgstr "Автообновление приостановлено (установлено на %s секунд)" -#, python-format +# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr, +# sr_Latn] +#, fuzzy, python-format msgid "Auto refresh paused - tab inactive (set to %s seconds)" msgstr "" +"Автообновление приостановлено — вкладка неактивна (установлено на %s " +"секунд)" Review Comment: <div> <div id="suggestion"> <div id="issue"><b>Wrong msgstr line split for python-format</b></div> <div id="fix"> The `msgstr` for `Auto refresh paused - tab inactive (set to %s seconds)` is incorrectly split across two msgstr lines (lines 2317 and 2318-2319). A Python format string placeholder (`%s`) must not be separated from its closing text across distinct msgstr lines — this breaks `python-format` extraction and causes runtime formatting errors. Merge the string into one msgstr line. </div> </div> <small><i>Code Review Run #c5b8fe</i></small> </div> --- Should Bito avoid suggestions like this for future reviews? (<a href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>) - [ ] Yes, avoid them -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
