-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Удаленение лишних переводов #224
Удаленение лишних переводов #224
Conversation
WalkthroughВнесены изменения в локализацию русскоязычной версии игры, включая удаление множества строк и описаний, связанных с различными игровыми функциями, такими как меню, команды администратора, взаимодействия с персонажами и химическими компонентами. Это привело к упрощению интерфейса и сокращению доступных опций для игроков и администраторов. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (54)
- Resources/Locale/ru-RU/HUD/game-hud.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/antag.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/commands/aghost.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/commands/panicbunker.ftl (1 hunks)
- Resources/Locale/ru-RU/administration/smites.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/ui/actions.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/ui/admin-menu-window.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/ui/tabs/panicbunker-tab.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/ui/tabs/player-tab.ftl (0 hunks)
- Resources/Locale/ru-RU/administration/ui/tabs/round-tab.ftl (0 hunks)
- Resources/Locale/ru-RU/anomaly/anomaly.ftl (0 hunks)
- Resources/Locale/ru-RU/borg/borg.ftl (0 hunks)
- Resources/Locale/ru-RU/cargo/bounties.ftl (0 hunks)
- Resources/Locale/ru-RU/cargo/cargo-bounty-console.ftl (0 hunks)
- Resources/Locale/ru-RU/cargo/cargo-console-component.ftl (0 hunks)
- Resources/Locale/ru-RU/character-appearance/components/magic-mirror-component.ftl (1 hunks)
- Resources/Locale/ru-RU/chat/commands/suicide-command.ftl (1 hunks)
- Resources/Locale/ru-RU/chat/managers/chat-manager.ftl (0 hunks)
- Resources/Locale/ru-RU/chat/ui/chat-box.ftl (0 hunks)
- Resources/Locale/ru-RU/chemistry/components/mixing-component.ftl (0 hunks)
- Resources/Locale/ru-RU/chemistry/components/reagent-dispenser-component.ftl (0 hunks)
- Resources/Locale/ru-RU/chemistry/components/solution-scanner-component.ftl (0 hunks)
- Resources/Locale/ru-RU/chemistry/components/solution-transfer-component.ftl (0 hunks)
- Resources/Locale/ru-RU/commands/stat-values-command.ftl (1 hunks)
- Resources/Locale/ru-RU/commands/toolshed-commands.ftl (0 hunks)
- Resources/Locale/ru-RU/communications/communications-console-component.ftl (0 hunks)
- Resources/Locale/ru-RU/components/power-monitoring-component.ftl (0 hunks)
- Resources/Locale/ru-RU/components/storage-component.ftl (0 hunks)
- Resources/Locale/ru-RU/construction/conditions/no-unstackable-in-tile.ftl (0 hunks)
- Resources/Locale/ru-RU/construction/construction-categories.ftl (0 hunks)
- Resources/Locale/ru-RU/construction/steps/arbitrary-insert-construction-graph-step.ftl (0 hunks)
- Resources/Locale/ru-RU/containers/containers.ftl (0 hunks)
- Resources/Locale/ru-RU/deltav/prototypes/access/accesses.ftl (0 hunks)
- Resources/Locale/ru-RU/devices/device-network.ftl (0 hunks)
- Resources/Locale/ru-RU/escape-menu/ui/escape-menu.ftl (0 hunks)
- Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl (0 hunks)
- Resources/Locale/ru-RU/fax/fax-admin.ftl (0 hunks)
- Resources/Locale/ru-RU/fax/fax.ftl (0 hunks)
- Resources/Locale/ru-RU/flavors/flavor-profiles.ftl (0 hunks)
- Resources/Locale/ru-RU/forensics/forensics.ftl (0 hunks)
- Resources/Locale/ru-RU/game-ticking/forcemap-command.ftl (0 hunks)
- Resources/Locale/ru-RU/game-ticking/game-presets/preset-traitor.ftl (0 hunks)
- Resources/Locale/ru-RU/game-ticking/game-ticker.ftl (0 hunks)
- Resources/Locale/ru-RU/ghost/ghost-gui.ftl (0 hunks)
- Resources/Locale/ru-RU/ghost/roles/ghost-role-component.ftl (0 hunks)
- Resources/Locale/ru-RU/guidebook/chemistry/conditions.ftl (0 hunks)
- Resources/Locale/ru-RU/guidebook/chemistry/core.ftl (0 hunks)
- Resources/Locale/ru-RU/guidebook/chemistry/effects.ftl (0 hunks)
- Resources/Locale/ru-RU/guidebook/guides.ftl (0 hunks)
- Resources/Locale/ru-RU/health-examinable/health-examinable-carbon.ftl (0 hunks)
- Resources/Locale/ru-RU/holiday/greet/holiday-greet.ftl (0 hunks)
- Resources/Locale/ru-RU/info/ban.ftl (0 hunks)
- Resources/Locale/ru-RU/info/rules.ftl (0 hunks)
- Resources/Locale/ru-RU/interaction/interaction-popup-component.ftl (0 hunks)
💤 Files with no reviewable changes (50)
- Resources/Locale/ru-RU/HUD/game-hud.ftl
- Resources/Locale/ru-RU/administration/antag.ftl
- Resources/Locale/ru-RU/administration/commands/aghost.ftl
- Resources/Locale/ru-RU/administration/smites.ftl
- Resources/Locale/ru-RU/administration/ui/actions.ftl
- Resources/Locale/ru-RU/administration/ui/admin-menu-window.ftl
- Resources/Locale/ru-RU/administration/ui/tabs/panicbunker-tab.ftl
- Resources/Locale/ru-RU/administration/ui/tabs/player-tab.ftl
- Resources/Locale/ru-RU/administration/ui/tabs/round-tab.ftl
- Resources/Locale/ru-RU/anomaly/anomaly.ftl
- Resources/Locale/ru-RU/borg/borg.ftl
- Resources/Locale/ru-RU/cargo/bounties.ftl
- Resources/Locale/ru-RU/cargo/cargo-bounty-console.ftl
- Resources/Locale/ru-RU/cargo/cargo-console-component.ftl
- Resources/Locale/ru-RU/chat/managers/chat-manager.ftl
- Resources/Locale/ru-RU/chat/ui/chat-box.ftl
- Resources/Locale/ru-RU/chemistry/components/mixing-component.ftl
- Resources/Locale/ru-RU/chemistry/components/reagent-dispenser-component.ftl
- Resources/Locale/ru-RU/chemistry/components/solution-scanner-component.ftl
- Resources/Locale/ru-RU/chemistry/components/solution-transfer-component.ftl
- Resources/Locale/ru-RU/commands/toolshed-commands.ftl
- Resources/Locale/ru-RU/communications/communications-console-component.ftl
- Resources/Locale/ru-RU/components/power-monitoring-component.ftl
- Resources/Locale/ru-RU/components/storage-component.ftl
- Resources/Locale/ru-RU/construction/conditions/no-unstackable-in-tile.ftl
- Resources/Locale/ru-RU/construction/construction-categories.ftl
- Resources/Locale/ru-RU/construction/steps/arbitrary-insert-construction-graph-step.ftl
- Resources/Locale/ru-RU/containers/containers.ftl
- Resources/Locale/ru-RU/deltav/prototypes/access/accesses.ftl
- Resources/Locale/ru-RU/devices/device-network.ftl
- Resources/Locale/ru-RU/escape-menu/ui/escape-menu.ftl
- Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl
- Resources/Locale/ru-RU/fax/fax-admin.ftl
- Resources/Locale/ru-RU/fax/fax.ftl
- Resources/Locale/ru-RU/flavors/flavor-profiles.ftl
- Resources/Locale/ru-RU/forensics/forensics.ftl
- Resources/Locale/ru-RU/game-ticking/forcemap-command.ftl
- Resources/Locale/ru-RU/game-ticking/game-presets/preset-traitor.ftl
- Resources/Locale/ru-RU/game-ticking/game-ticker.ftl
- Resources/Locale/ru-RU/ghost/ghost-gui.ftl
- Resources/Locale/ru-RU/ghost/roles/ghost-role-component.ftl
- Resources/Locale/ru-RU/guidebook/chemistry/conditions.ftl
- Resources/Locale/ru-RU/guidebook/chemistry/core.ftl
- Resources/Locale/ru-RU/guidebook/chemistry/effects.ftl
- Resources/Locale/ru-RU/guidebook/guides.ftl
- Resources/Locale/ru-RU/health-examinable/health-examinable-carbon.ftl
- Resources/Locale/ru-RU/holiday/greet/holiday-greet.ftl
- Resources/Locale/ru-RU/info/ban.ftl
- Resources/Locale/ru-RU/info/rules.ftl
- Resources/Locale/ru-RU/interaction/interaction-popup-component.ftl
🔇 Additional comments (6)
Resources/Locale/ru-RU/character-appearance/components/magic-mirror-component.ftl (1)
2-2
: Перевод корректен.Оставшийся перевод для заголовка окна волшебного зеркала выглядит правильным и соответствует оригиналу.
Resources/Locale/ru-RU/commands/stat-values-command.ftl (3)
Line range hint
1-16
: Удаление неиспользуемых переводов одобрено.Удаление строк, связанных со статистикой частоты отрисовки (
stat-drawrate-values
,stat-drawrate-id
,stat-drawrate-rate
), соответствует цели PR по удалению ненужных переводов. Это изменение не должно отрицательно повлиять на функциональность игры.
Line range hint
1-17
: Общая оценка изменений: ОдобреноВнесенные изменения соответствуют целям PR и улучшают качество русской локализации:
- Удалены неиспользуемые переводы, связанные со статистикой частоты отрисовки.
- Исправлен перевод
stat-item-price
для более точного отражения контекста.- Оставшиеся переводы сохранены и выглядят корректно.
Эти изменения должны привести к более чёткому и понятному интерфейсу для русскоязычных игроков.
17-17
: Одобрено изменение перевода, рекомендуется проверка связанных строк.Изменение перевода
stat-item-price
с "Цена" на "Размер" является корректным и лучше соответствует контексту статистики предметов.Рекомендуется проверить связанные переводы, чтобы убедиться в согласованности терминологии. Выполните следующий скрипт для поиска связанных строк:
✅ Verification successful
Изменение перевода подтверждено.
Изменение перевода
stat-item-price
с "Цена" на "Размер" соответствует использованию термина в контексте статистики предметов и согласуется с другими переводами.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Поиск связанных переводов для проверки согласованности # Поиск строк, содержащих "item" и "size" или "размер" rg --type ftl -i "item.*(size|размер)" Resources/Locale/ru-RU/Length of output: 92
Script:
#!/bin/bash # Описание: Поиск связанных переводов для проверки согласованности # Поиск файлов с расширением .ftl и строк, содержащих "item" и "size" или "размер" rg -i "item.*(size|размер)" --glob "*.ftl" Resources/Locale/ru-RU/Length of output: 1447
Resources/Locale/ru-RU/chat/commands/suicide-command.ftl (1)
Line range hint
1-9
: Рассмотрите возможность сохранения удаленных строк локализацииУдаление строк
suicide-command-no-mind
иsuicide-command-denied
может привести к потере важной обратной связи для игроков в определенных ситуациях. Это может вызвать путаницу, когда команда не срабатывает без объяснения причины.Рекомендуется:
- Сохранить эти строки, если соответствующая функциональность все еще присутствует в игре.
- Если функциональность изменилась, рассмотрите возможность обновления этих строк вместо их полного удаления.
Предлагаю вернуть удаленные строки:
suicide-command-default-text-self = Вы пытаетесь прикусить свой собственный язык! +suicide-command-no-mind = У вас нет разума, чтобы совершить самоубийство! +suicide-command-denied = Вы не можете совершить самоубийство прямо сейчас! suicide-command-already-dead = Вы не можете совершить самоубийство. Вы мертвы.Для проверки использования этих строк в коде выполните следующий скрипт:
Если скрипт не найдет использования этих строк, возможно, соответствующая функциональность была удалена из кода. В таком случае, удаление строк локализации оправдано.
✅ Verification successful
Удаленные строки локализации не используются в кодовой базе
Проверка показала, что строки
suicide-command-no-mind
иsuicide-command-denied
нигде не используются в проекте. Таким образом, их удаление не повлияет на функциональность или опыт пользователей.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Проверка использования удаленных строк локализации в коде # Поиск использования `suicide-command-no-mind` echo "Поиск использования 'suicide-command-no-mind':" rg --type csharp "suicide-command-no-mind" # Поиск использования `suicide-command-denied` echo "Поиск использования 'suicide-command-denied':" rg --type csharp "suicide-command-denied"Length of output: 300
Script:
#!/bin/bash # Описание: Полная проверка использования удаленных строк локализации во всем коде # Поиск использования `suicide-command-no-mind` echo "Поиск использования 'suicide-command-no-mind' во всех файлах:" rg "suicide-command-no-mind" # Поиск использования `suicide-command-denied` echo "Поиск использования 'suicide-command-denied' во всех файлах:" rg "suicide-command-denied"Length of output: 332
Resources/Locale/ru-RU/administration/commands/panicbunker.ftl (1)
28-28
: Одобрено: Упрощение команд паник-бункераИзменения упрощают структуру команд, удаляя неиспользуемые переводы для минут и оставляя только настройку в часах. Это делает интерфейс более понятным и удобным для администраторов.
Давайте проверим, действительно ли удаленные переводы больше не используются:
VIVA LA LOST PARADISE!
Описание обновления:
Удаление лишних переводов
Медиа
Проверки
Изменения
🆑 Farrellka