White changelog #133
White changelog #133
Conversation
ПроцессВ классе Изменения
Возможно связанные PR
Предложенные метки
Предложенные рецензенты
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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (3)
Content.Client/Lobby/LobbyState.cs (2)
Line range hint
244-256: Рассмотрите добавление конфигурируемого количества записейИзменение для использования "WhiteChangelog" корректно реализовано, но жёстко заданное ограничение в 3 записи может быть недостаточным для пользователей. Рекомендуется сделать это значение настраиваемым через конфигурацию.
- .Take(3); + .Take(_configurationManager.GetCVar(CCVars.LobbyChangelogEntries));
Line range hint
244-256: Необходимо локализовать текст интерфейсаТекст "White Dream" жёстко закодирован на английском языке. Рекомендуется:
- Перенести текст в локализационные файлы
- Вынести настройки шрифта в стили UI
-Lobby!.LabelName.SetMarkup("[font=\"Bedstead\" size=20] White Dream [/font]"); +Lobby!.LabelName.SetMarkup(Loc.GetString("lobby-label-name"));Resources/Changelog/WhiteChangelog.yml (1)
1-846: Рекомендуется добавить валидацию схемы YAMLДля обеспечения корректности формата changelog'а рекомендуется добавить схему YAML и валидацию при коммите.
Хотите, чтобы я помог создать схему YAML и настроить валидацию?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (5)
Content.Client/Lobby/LobbyState.cs(2 hunks)Resources/Changelog/Changelog.yml(0 hunks)Resources/Changelog/WhiteChangelog.yml(1 hunks)Resources/Locale/en-US/_white/changelog/changelog-window.ftl(1 hunks)Resources/Locale/ru-RU/_white/changelog/changelog-window.ftl(1 hunks)
💤 Files with no reviewable changes (1)
- Resources/Changelog/Changelog.yml
✅ Files skipped from review due to trivial changes (2)
- Resources/Locale/ru-RU/_white/changelog/changelog-window.ftl
- Resources/Locale/en-US/_white/changelog/changelog-window.ftl
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
Resources/Changelog/WhiteChangelog.yml(1 hunks)
🔇 Additional comments (3)
Resources/Changelog/WhiteChangelog.yml (3)
1-861: Необходимо обеспечить консистентность записей
В файле обнаружены те же проблемы, что были отмечены ранее:
- Некоторые записи содержат только русский текст без английского перевода
- Формат двуязычных сообщений не унифицирован
1-861: Проверьте хронологический порядок записей
Все записи должны быть отсортированы по времени в порядке убывания (от новых к старым). Рекомендуется проверить правильность сортировки.
790-814: 🛠️ Refactor suggestion
Используйте двуязычный формат для всех записей
Записи с ID 89 содержат только русский текст. Рекомендуется добавить английские переводы и использовать единый формат:
- type: Tweak
- message: Швабра теперь моет весь тайл
+ message: >-
+ Mop now cleans the entire tile / Швабра теперь моет весь тайлLikely invalid or redundant comment.
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Описание PR
Отдельный ченжлог для контента вайта. Нужно поменять CHANGELOG_DIR на WhiteChangelog
Изменения
🆑