Баланс черт, добавление новой, разрешение вульпам #322
Conversation
WalkthroughВ данном запросе на изменение добавлен новыйTrait "Медленный едок" (Slow Eater) в локализацию игры "Lost Paradise" на английском и русском языках. Trait описывает, что персонаж ест и пьет вдвое медленнее обычного. Также в файле Changes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used📓 Path-based instructions (1)Resources/Locale/en-US/_LostParadise/traits/traits.ftl (1)Pattern А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix 🔇 Additional comments (2)Resources/Locale/en-US/_LostParadise/traits/traits.ftl (2)
Название "Slow Eater" правильно форматировано и соответствует стандартам локализации.
Описание четко объясняет механику черты и следует рекомендациям по форматированию. Предыдущие замечания по объединению предложений уже учтены в текущей версии. 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 (
|
github-actions
bot
added
Статус: Требуется одобрение
|
@Evgencheg как отменить все коммиты? На этот пр со старого всё перенеслось почему-то, я так и не понял |
|
эээ какой ошибка ямл, ты чё, френдли фаер, только что не было |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (6)
Resources/Locale/en-US/_LostParadise/traits/traits.ftl(1 hunks)Resources/Locale/en-US/traits/traits.ftl(1 hunks)Resources/Locale/ru-RU/_LostParadise/traits/traits.ftl(1 hunks)Resources/Locale/ru-RU/traits/traits.ftl(1 hunks)Resources/Prototypes/Traits/skills.yml(2 hunks)Resources/Prototypes/_LostParadise/Traits/disabilities.yml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (6)
Resources/Locale/en-US/_LostParadise/traits/traits.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/en-US/traits/traits.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/_LostParadise/traits/traits.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/traits/traits.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/Traits/skills.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Traits/disabilities.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
🪛 yamllint
Resources/Prototypes/_LostParadise/Traits/disabilities.yml
[error] 18-18: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (8)
Resources/Locale/en-US/_LostParadise/traits/traits.ftl (1)
1-1: Корректное определение названия черты!
Название черты следует стандартному формату именования и хорошо описывает функциональность.
Resources/Locale/ru-RU/_LostParadise/traits/traits.ftl (1)
1-1: Корректное именование трейта!
Название трейта соответствует формату локализации и хорошо описывает его суть.
Resources/Prototypes/_LostParadise/Traits/disabilities.yml (3)
1-4: 🧹 Nitpick (assertive)
Проверьте баланс очков черты.
Учитывая, что черта значительно замедляет потребление (в 1.6 раза), стоимость в 1 очко может быть занижена. Рекомендуется рассмотреть увеличение стоимости до 2 очков для лучшего баланса.
18-18: 🧹 Nitpick (assertive)
Добавьте пустую строку в конец файла.
Отсутствует перевод строки в конце файла, что может вызвать проблемы при обработке файла некоторыми инструментами.
- MedicalBorg
+Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint
[error] 18-18: no new line character at the end of file
(new-line-at-end-of-file)
6-8:
Проверьте множители задержки.
Множители 1.6 для еды и напитков не соответствуют описанию в локализации, где указано "вдвое медленнее". Рекомендуется:
- foodDelayMultiplier: 1.6
- drinkDelayMultiplier: 1.6
+ foodDelayMultiplier: 2.0
+ drinkDelayMultiplier: 2.0Likely invalid or redundant comment.
Resources/Prototypes/Traits/skills.yml (2)
94-94: Подтверждаю изменение множителя времени раздевания.
Уменьшение множителя stripTimeMultiplier с 0.7 до 0.6 для черты "Бывалый вор" корректно реализует заявленное ускорение процесса раздевания.
112-113: Подтверждаю балансировку множителей потребления.
Изменение множителей foodDelayMultiplier и drinkDelayMultiplier с 0.4 на 0.5 для черты "Прожорливый" является частью общего баланса черт, связанных с приёмом пищи, и хорошо сочетается с добавлением новой черты "Медленный едок".
Resources/Locale/ru-RU/traits/traits.ftl (1)
144-147: Перевод выглядит корректно!
Перевод нового трейта "SlowEater" соответствует стилю других переводов в файле и хорошо сочетается с существующим трейтом "Voracious" (Прожорливый), представляя противоположный эффект.
| trait-description-LPPSlowEater = | ||
| Whether due to personal habits or physical limitations, you take your time with meals. | ||
| You consume food and drinks at half the normal speed. |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Описание черты выглядит правильно, но есть предложение по улучшению.
Описание корректно объясняет механику, но для лучшей согласованности с другими чертами, предлагаю объединить два предложения в одно.
Предлагаемое изменение:
trait-description-LPPSlowEater =
- Whether due to personal habits or physical limitations, you take your time with meals.
- You consume food and drinks at half the normal speed.
+ Whether due to personal habits or physical limitations, you take your time with meals, consuming food and drinks at half the normal speed.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| trait-description-LPPSlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals. | |
| You consume food and drinks at half the normal speed. | |
| trait-description-LPPSlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals, consuming food and drinks at half the normal speed. |
| trait-description-LPPSlowEater = | ||
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | ||
| Вы потребляете еду и напитки в два раза медленнее, чем обычно. |
There was a problem hiding this comment.
Несоответствие множителя в описании
В описании указано, что потребление происходит "в два раза медленнее", однако в реализации трейта используется множитель 1.6. Необходимо привести описание в соответствие с фактической механикой.
Предлагаемое исправление:
trait-name-LPPSlowEater = Медленный едок
trait-description-LPPSlowEater =
Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные.
- Вы потребляете еду и напитки в два раза медленнее, чем обычно.
+ Вы потребляете еду и напитки в полтора раза медленнее, чем обычно.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| trait-description-LPPSlowEater = | |
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | |
| Вы потребляете еду и напитки в два раза медленнее, чем обычно. | |
| trait-description-LPPSlowEater = | |
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | |
| Вы потребляете еду и напитки в полтора раза медленнее, чем обычно. |
| - !type:CharacterSpeciesRequirement | ||
| inverted: true | ||
| species: | ||
| - IPC | ||
| - !type:CharacterJobRequirement | ||
| inverted: true | ||
| jobs: | ||
| - Borg | ||
| - MedicalBorg |
There was a problem hiding this comment.
💡 Codebase verification
Список исключений требует дополнения.
В текущем списке исключений отсутствуют некоторые синтетические расы и профессии, которые также не должны иметь эту черту характера. Рекомендуется добавить:
- Skeleton (скелеты не едят)
- Terminator (синтетические существа)
🔗 Analysis chain
Подтвердите список исключений.
Текущие исключения логичны (IPC и борги не едят), но рекомендуется проверить, не нужно ли добавить другие виды синтетиков или роботов в список исключений.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск других синтетических рас или профессий
rg -l "synthetic|robot|android" Resources/Prototypes/*/Species/
Length of output: 112
Script:
#!/bin/bash
# Поиск всех файлов с определениями видов (species)
fd -e yml -e yaml species Resources/Prototypes
# Поиск всех файлов с определениями профессий (jobs)
fd -e yml -e yaml jobs Resources/Prototypes
# Поиск упоминаний синтетиков, роботов и андроидов в этих файлах
rg -i "synthetic|robot|android|cyborg|borg|artificial|machine" Resources/Prototypes/**/*.yml
Length of output: 50834
🧰 Tools
🪛 yamllint
[error] 18-18: no new line character at the end of file
(new-line-at-end-of-file)
| <<<<<<< HEAD | ||
| ======= | ||
| trait-name-SlowEater = Slow Eater | ||
| trait-description-SlowEater = | ||
| Whether due to personal habits or physical limitations, you take your time with meals. | ||
| You consume food and drinks at half the normal speed. | ||
|
|
||
| >>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d |
There was a problem hiding this comment.
Критическая ошибка: Необходимо разрешить конфликт слияния!
В файле присутствуют неразрешенные маркеры конфликта слияния (<<<<<<< HEAD, =======, >>>>>>>). Их необходимо удалить, сохранив нужные изменения.
-<<<<<<< HEAD
-=======
trait-name-SlowEater = Slow Eater
trait-description-SlowEater =
Whether due to personal habits or physical limitations, you take your time with meals.
You consume food and drinks at half the normal speed.
->>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <<<<<<< HEAD | |
| ======= | |
| trait-name-SlowEater = Slow Eater | |
| trait-description-SlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals. | |
| You consume food and drinks at half the normal speed. | |
| >>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d | |
| trait-name-SlowEater = Slow Eater | |
| trait-description-SlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals. | |
| You consume food and drinks at half the normal speed. |
| trait-name-SlowEater = Slow Eater | ||
| trait-description-SlowEater = | ||
| Whether due to personal habits or physical limitations, you take your time with meals. | ||
| You consume food and drinks at half the normal speed. | ||
|
|
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Форматирование и содержание соответствует стандартам!
Новая черта "Slow Eater" корректно интегрирована в существующую структуру файла и хорошо дополняет существующую черту "Voracious".
Рекомендация по улучшению описания
Для поддержания единообразия с другими чертами, предлагаю добавить конкретное числовое значение в описание.
trait-description-SlowEater =
Whether due to personal habits or physical limitations, you take your time with meals.
- You consume food and drinks at half the normal speed.
+ You consume food and drinks at 50% of the normal speed.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| trait-name-SlowEater = Slow Eater | |
| trait-description-SlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals. | |
| You consume food and drinks at half the normal speed. | |
| trait-name-SlowEater = Slow Eater | |
| trait-description-SlowEater = | |
| Whether due to personal habits or physical limitations, you take your time with meals. | |
| You consume food and drinks at 50% of the normal speed. |
| <<<<<<< HEAD | ||
| ======= | ||
| trait-name-SlowEater = Медленный едок | ||
| trait-description-SlowEater = | ||
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | ||
| Вы потребляете еду и напитки в два раза медленнее, чем обычно. | ||
| >>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d |
There was a problem hiding this comment.
Необходимо разрешить конфликт слияния!
В файле остались маркеры конфликта слияния Git, которые необходимо удалить:
-<<<<<<< HEAD
-
-=======
trait-name-SlowEater = Медленный едок
trait-description-SlowEater =
Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные.
Вы потребляете еду и напитки в два раза медленнее, чем обычно.
->>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <<<<<<< HEAD | |
| ======= | |
| trait-name-SlowEater = Медленный едок | |
| trait-description-SlowEater = | |
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | |
| Вы потребляете еду и напитки в два раза медленнее, чем обычно. | |
| >>>>>>> 4298b63e04503b7af7a74458204edcb96c86031d | |
| trait-name-SlowEater = Медленный едок | |
| trait-description-SlowEater = | |
| Будь то врождённая проблема со здоровьем, дисфункция ротовых органов и мышц или даже ваш собственный способ наслаждения пищей, вы едите медленнее, чем остальные. | |
| Вы потребляете еду и напитки в два раза медленнее, чем обычно. |
@IAmLePacko Потому что блять 1 пр 1 бранч а не 2 пра 1 бранч бля |
Блокед мессаге. |
ТЫ НЕ пофиксил что я просил. Пока не починишь - не приму |
|
@Evgencheg я сделал вроде |
|
@Evgencheg проверь, а так тут уже весит моё одобрение |
Описание обновления:
В общем, да, всё как в названии.
Проверки
🆑 IAmLePacko