Halloween things2 #301
Halloween things2 #301
Conversation
WalkthroughВ данном пулл-реквесте внесены изменения в локализацию и определения сущностей, связанные с Хэллоуином, в русскоязычной версии игры или приложения. Удалены дублирующие записи для конфет, изменены идентификаторы для украшений в виде деревьев, добавлены новые элементы декора, а также обновлены свойства существующих сущностей. В результате улучшена структура и разнообразие элементов, связанных с Хэллоуином. Changes
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
Статус: Требуется одобрение
|
RSI Diff Bot; head commit 3128bc3 merging into 3657bfd Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 11
🧹 Outside diff range comments (1)
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/Flora/halloween.yml (1)
Line range hint
1-38: Рекомендация по оптимизации повторяющегося описанияВсе четыре сущности имеют идентичное описание. Предлагаю перенести общие поля
nameиdescriptionв родительскую сущностьBaseTreeдля уменьшения дублирования кода.Пример рефакторинга:
- type: entity parent: BaseTree id: LPPHalloweenTree01 - name: dead tree - description: Dead tree with leaves on it, looks creepy. components: type: Sprite sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi state: halloweentree01И добавить в родительскую сущность:
- type: entity id: BaseTree abstract: true name: dead tree description: Dead tree with leaves on it, looks creepy.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
⛔ Files ignored due to path filters (7)
Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/plant1.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/plant2.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/plant3.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/plant4.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumkin_minis.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumkin_minis2.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumkins.pngis excluded by!**/*.png
📒 Files selected for processing (8)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/consumable/food/halloween.ftl (1 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/decoration/flora/halloween.ftl (1 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/decoration/halloween.ftl (1 hunks)
- Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/Flora/halloween.yml (4 hunks)
- Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (2 hunks)
- Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml (2 hunks)
- Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (7)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/consumable/food/halloween.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/decoration/flora/halloween.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/decoration/halloween.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/Flora/halloween.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🪛 yamllint
Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml
[error] 10-10: wrong indentation: expected at least 5
(indentation)
[error] 11-11: too many spaces inside brackets
(brackets)
[error] 11-11: too many spaces inside brackets
(brackets)
[error] 15-15: trailing spaces
(trailing-spaces)
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
[error] 18-18: trailing spaces
(trailing-spaces)
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml
[error] 8-8: wrong indentation: expected 4 but found 2
(indentation)
[error] 12-12: wrong indentation: expected 6 but found 4
(indentation)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 45-45: wrong indentation: expected 4 but found 2
(indentation)
[error] 50-50: wrong indentation: expected 6 but found 4
(indentation)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 63-63: wrong indentation: expected 6 but found 4
(indentation)
[error] 67-67: wrong indentation: expected 8 but found 6
(indentation)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 79-79: wrong indentation: expected 4 but found 2
(indentation)
[error] 86-86: wrong indentation: expected 6 but found 4
(indentation)
[error] 90-90: wrong indentation: expected 8 but found 6
(indentation)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 101-101: wrong indentation: expected 4 but found 2
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 116-116: wrong indentation: expected 4 but found 2
(indentation)
[error] 123-123: wrong indentation: expected 4 but found 2
(indentation)
[error] 125-125: too many blank lines
(1 > 0) (empty-lines)
🔇 Additional comments (11)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/decoration/halloween.ftl (1)
9-14: Отличная реализация повторяющихся переводов!Правильное использование ссылок на базовый перевод через синтаксис { entity } как для названия, так и для описания.
Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml (3)
Line range hint
1-4: Базовое определение сущности корректно!Структура и описание соответствуют стандартам.
🧰 Tools
🪛 yamllint
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 10-10: wrong indentation: expected at least 5
(indentation)
[error] 11-11: too many spaces inside brackets
(brackets)
[error] 11-11: too many spaces inside brackets
(brackets)
[error] 15-15: trailing spaces
(trailing-spaces)
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
[error] 18-18: trailing spaces
(trailing-spaces)
12-18: Эффективная реализация случайных спрайтов!Объединение нескольких сущностей в одну с случайными спрайтами - хорошее решение для оптимизации.
#!/bin/bash # Description: Verify that old candy entities are completely removed rg "LPPCandy[2-4]|LPPLolipop" --type yaml🧰 Tools
🪛 yamllint
[error] 15-15: trailing spaces
(trailing-spaces)
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
[error] 18-18: trailing spaces
(trailing-spaces)
9-11: Проверить наличие состояний спрайтовУбедитесь, что все указанные состояния спрайтов (
candy_1) присутствуют в файле_LostParadise/Objects/Consumable/halloween.rsi.✅ Verification successful
Состояние спрайта подтверждено
Состояние спрайта
candy_1найдено в файлеResources/Textures/_LostParadise/Objects/Consumable/halloween.rsi/meta.json, что подтверждает его корректное определение в конфигурации.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check if the RSI file exists and contains the required sprite states rg -g "*.rsi/**" -l "candy_1"Length of output: 137
Script:
#!/bin/bash # Search for RSI metadata files that might contain sprite state definitions fd -e "meta.json" -p ".*halloween.*\.rsi/meta\.json$" # Search for any files containing candy-related sprite states rg -g "*.json" "candy_1" # Search for halloween-related RSI directories fd -t d ".rsi$" -p ".*halloween.*"Length of output: 322
🧰 Tools
🪛 yamllint
[error] 10-10: wrong indentation: expected at least 5
(indentation)
[error] 11-11: too many spaces inside brackets
(brackets)
[error] 11-11: too many spaces inside brackets
(brackets)
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/Flora/halloween.yml (1)
3-3: Корректное изменение идентификаторов сущностейПрефикс
LPPдобавлен корректно ко всем идентификаторам деревьев, что соответствует принятым соглашениям об именовании.Also applies to: 13-13, 23-23, 33-33
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml (2)
32-32: Изменение свойства noRot одобрено.Разрешение вращения для декоративных флажков логично с точки зрения геймплея, так как позволит игрокам более гибко размещать украшения.
47-47: Проверьте влияние вращения на освещение.Изменение свойства noRot логично, но так как объект является источником света, убедитесь, что вращение не создаст проблем с компонентом PointLight.
✅ Verification successful
Вращение источника света безопасно
Проверка показала, что объект с PointLight является настенной гирляндой с небольшим радиусом освещения (2.0). В данном случае вращение не создаст проблем, так как:
- Радиус освещения небольшой
- Источник света симметричный (гирлянда)
- Энергия света умеренная (2.5)
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Поиск других сущностей с комбинацией PointLight и noRot: false ast-grep --pattern $'components: $$$ - type: PointLight $$$ - type: Sprite $$$ noRot: false'Length of output: 111
Script:
#!/bin/bash # Поиск файла halloween.yml и проверка его содержимого fd halloween.yml --exec cat {} # Поиск всех yaml файлов с PointLight rg -l "type: PointLight" --type yamlLength of output: 17202
Script:
#!/bin/bash # Поиск всех сущностей с PointLight в halloween.yml с контекстом rg "type: PointLight" -A 10 Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.ymlLength of output: 302
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (4)
5-15: Изменения в LPPPSpiderDeco выглядят корректно!Добавление компонента Clickable и настройка размещения улучшают функциональность декорации.
🧰 Tools
🪛 yamllint
[error] 8-8: wrong indentation: expected 4 but found 2
(indentation)
[error] 12-12: wrong indentation: expected 6 but found 4
(indentation)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 13-13: too many spaces inside brackets
(brackets)
96-124: Отличная реализация увядших растений!Хорошее использование наследования для создания вариаций растений. Структура кода чистая и эффективная.
🧰 Tools
🪛 yamllint
[error] 101-101: wrong indentation: expected 4 but found 2
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 116-116: wrong indentation: expected 4 but found 2
(indentation)
[error] 123-123: wrong indentation: expected 4 but found 2
(indentation)
73-75:⚠️ Potential issueТа же опечатка в названии большой тыквы
Аналогичная опечатка в названии: "pumkin" должно быть "pumpkin"
Предлагаемые изменения:
id: LPPPPumpkinBigDeco - name: pumkin + name: pumpkinLikely invalid or redundant comment.
Line range hint
1-125: Необходимо проверить состояния спрайтовДавайте убедимся, что все используемые состояния спрайтов определены в RSI файле.
✅ Verification successful
Все состояния спрайтов корректно определены
Проверка показала, что все используемые состояния спрайтов присутствуют в RSI файле:
- Базовые состояния:
pumkins,plant1,plant2,plant3,plant4- Состояния тыкв:
pumkin_minis,pumkin_minis2- Состояния пауков:
spider_big,spider_big2,spider_mini1,spider_mini2,spider_mini3🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that all sprite states referenced in the YAML exist in the RSI meta.json # Extract all state references from the YAML STATES=$(cat "Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml" | grep "state:" | awk '{print $2}') # Check if these states exist in the RSI meta.json for state in $STATES; do echo "Checking state: $state" rg -A 1 "\"$state\"" "Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json" doneLength of output: 1625
Script:
#!/bin/bash # Let's also check the RandomSprite states grep -A 2 "available:" "Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml" | grep -v "available:" | grep -v "-" | awk '{print $1}' | while read -r state; do echo "Checking RandomSprite state: $state" rg "\"$state\"" "Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json" done # Also verify pumkin_minis states echo "Checking pumkin_minis states:" rg "pumkin_minis" "Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json" # And spider states echo "Checking spider states:" rg "spider_" "Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json"Length of output: 1133
🧰 Tools
🪛 yamllint
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 45-45: wrong indentation: expected 4 but found 2
(indentation)
[error] 50-50: wrong indentation: expected 6 but found 4
(indentation)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 63-63: wrong indentation: expected 6 but found 4
(indentation)
[error] 67-67: wrong indentation: expected 8 but found 6
(indentation)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 79-79: wrong indentation: expected 4 but found 2
(indentation)
[error] 86-86: wrong indentation: expected 6 but found 4
(indentation)
[error] 90-90: wrong indentation: expected 8 but found 6
(indentation)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 101-101: wrong indentation: expected 4 but found 2
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 116-116: wrong indentation: expected 4 but found 2
(indentation)
[error] 123-123: wrong indentation: expected 4 but found 2
(indentation)
[error] 125-125: too many blank lines
(1 > 0) (empty-lines)
| .desc = { ent-LPPCandy.desc } | ||
| ent-LPPLolipop = конфета на палочке | ||
| .desc = Всё такая же конфета, но на палочке. | ||
| .desc = Обычная конфета состоящия из сахара и других вещей |
There was a problem hiding this comment.
Исправьте грамматические ошибки и пунктуацию.
В описании присутствуют следующие ошибки:
- "состоящия" -> "состоящая" (грамматическая ошибка)
- Отсутствует точка в конце предложения
- .desc = Обычная конфета состоящия из сахара и других вещей
+ .desc = Обычная конфета, состоящая из сахара и других вещей.📝 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.
| .desc = Обычная конфета состоящия из сахара и других вещей | |
| .desc = Обычная конфета, состоящая из сахара и других вещей. |
| @@ -1,10 +1,2 @@ | |||
| ent-LPPCandy = конфета | |||
There was a problem hiding this comment.
Исправьте регистр первой буквы в переводе.
Согласно правилам кодирования, все переводы, начинающиеся с "ent-", должны начинаться с заглавной буквы после знака равенства.
-ent-LPPCandy = конфета
+ent-LPPCandy = Конфета📝 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.
| ent-LPPCandy = конфета | |
| ent-LPPCandy = Конфета |
| ent-LPPHalloweenTree01 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree02 = мёртвое дерево | ||
| ent-LPPHalloweenTree02 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree03 = мёртвое дерево | ||
| ent-LPPHalloweenTree03 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree04 = мёртвое дерево | ||
| ent-LPPHalloweenTree04 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Рекомендуется использовать ссылки на повторяющиеся переводы
Все четыре сущности имеют идентичные переводы. Согласно правилам, повторяющиеся переводы должны использовать синтаксис ссылок.
Рекомендуется реорганизовать код следующим образом:
ent-LPPHalloweenTree01 = Мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree02 = мёртвое дерево
- .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree03 = мёртвое дерево
- .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree04 = мёртвое дерево
- .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
+ent-LPPHalloweenTree02 = { ent-LPPHalloweenTree01 }
+ .desc = { ent-LPPHalloweenTree01.desc }
+ent-LPPHalloweenTree03 = { ent-LPPHalloweenTree01 }
+ .desc = { ent-LPPHalloweenTree01.desc }
+ent-LPPHalloweenTree04 = { ent-LPPHalloweenTree01 }
+ .desc = { ent-LPPHalloweenTree01.desc }📝 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.
| ent-LPPHalloweenTree01 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree02 = мёртвое дерево | |
| ent-LPPHalloweenTree02 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree03 = мёртвое дерево | |
| ent-LPPHalloweenTree03 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree04 = мёртвое дерево | |
| ent-LPPHalloweenTree04 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree01 = Мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree02 = { ent-LPPHalloweenTree01 } | |
| .desc = { ent-LPPHalloweenTree01.desc } | |
| ent-LPPHalloweenTree03 = { ent-LPPHalloweenTree01 } | |
| .desc = { ent-LPPHalloweenTree01.desc } | |
| ent-LPPHalloweenTree04 = { ent-LPPHalloweenTree01 } | |
| .desc = { ent-LPPHalloweenTree01.desc } |
Необходимо исправить регистр текста после знака "="
Согласно правилам кодирования, все переводы, начинающиеся с "ent-", должны иметь текст с большой буквы после знака "=".
Примените следующие изменения:
-ent-LPPHalloweenTree01 = мёртвое дерево
+ent-LPPHalloweenTree01 = Мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree02 = мёртвое дерево
+ent-LPPHalloweenTree02 = Мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree03 = мёртвое дерево
+ent-LPPHalloweenTree03 = Мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
-ent-LPPHalloweenTree04 = мёртвое дерево
+ent-LPPHalloweenTree04 = Мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.📝 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.
| ent-LPPHalloweenTree01 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree02 = мёртвое дерево | |
| ent-LPPHalloweenTree02 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree03 = мёртвое дерево | |
| ent-LPPHalloweenTree03 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-HalloweenTree04 = мёртвое дерево | |
| ent-LPPHalloweenTree04 = мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree01 = Мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree02 = Мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree03 = Мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | |
| ent-LPPHalloweenTree04 = Мёртвое дерево | |
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. |
| ent-LPPPPumpkinBigDeco = тыква | ||
| .desc = Декоративные тыквы разных размеров. |
There was a problem hiding this comment.
Исправьте регистр первой буквы в переводе.
Аналогичная проблема с регистром первой буквы после знака =.
-ent-LPPPPumpkinBigDeco = тыква
+ent-LPPPPumpkinBigDeco = Тыква📝 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.
| ent-LPPPPumpkinBigDeco = тыква | |
| .desc = Декоративные тыквы разных размеров. | |
| ent-LPPPPumpkinBigDeco = Тыква | |
| .desc = Декоративные тыквы разных размеров. |
| ent-LPPPPumpkinDeco = тыква | ||
| .desc = Маленькая декоративная тыква. |
There was a problem hiding this comment.
Исправьте регистр первой буквы в переводе.
Согласно правилам локализации, все переводы, начинающиеся с 'ent-', должны начинаться с заглавной буквы после знака =.
-ent-LPPPPumpkinDeco = тыква
+ent-LPPPPumpkinDeco = Тыква📝 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.
| ent-LPPPPumpkinDeco = тыква | |
| .desc = Маленькая декоративная тыква. | |
| ent-LPPPPumpkinDeco = Тыква | |
| .desc = Маленькая декоративная тыква. |
| ent-LPPPottedPlantHalloween1 = растение в горшке | ||
| .desc = Высохшее растение в горшке, выглядит не особо живым. |
There was a problem hiding this comment.
Исправьте регистр первой буквы в переводе.
Та же проблема с регистром первой буквы после знака =.
-ent-LPPPottedPlantHalloween1 = растение в горшке
+ent-LPPPottedPlantHalloween1 = Растение в горшке📝 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.
| ent-LPPPottedPlantHalloween1 = растение в горшке | |
| .desc = Высохшее растение в горшке, выглядит не особо живым. | |
| ent-LPPPottedPlantHalloween1 = Растение в горшке | |
| .desc = Высохшее растение в горшке, выглядит не особо живым. |
| @@ -21,6 +21,28 @@ | |||
| }, | |||
| { | |||
| "name": "spider_mini3" | |||
| }, | |||
| { | |||
| "name": "pumkin_minis" | |||
There was a problem hiding this comment.
Исправьте опечатку в названии состояний
В названиях состояний присутствует опечатка: "pumkin" вместо правильного "pumpkin".
Примените следующие изменения:
- "name": "pumkin_minis"
+ "name": "pumpkin_minis"
- "name": "pumkin_minis2"
+ "name": "pumpkin_minis2"Also applies to: 29-29
| { | ||
| "name": "plant1" | ||
| }, | ||
| { | ||
| "name": "plant2" | ||
| }, | ||
| { | ||
| "name": "plant3" | ||
| }, | ||
| { | ||
| "name": "plant4" |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Рекомендация по улучшению именования растений
Текущие имена состояний (plant1, plant2, plant3, plant4) слишком общие. Рекомендуется использовать более описательные названия, отражающие внешний вид или тематику растений.
Пример более информативных названий:
- "name": "plant1"
+ "name": "halloween_plant_thorny"
- "name": "plant2"
+ "name": "halloween_plant_twisted"
- "name": "plant3"
+ "name": "halloween_plant_dead"
- "name": "plant4"
+ "name": "halloween_plant_spooky"📝 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.
| { | |
| "name": "plant1" | |
| }, | |
| { | |
| "name": "plant2" | |
| }, | |
| { | |
| "name": "plant3" | |
| }, | |
| { | |
| "name": "plant4" | |
| { | |
| "name": "halloween_plant_thorny" | |
| }, | |
| { | |
| "name": "halloween_plant_twisted" | |
| }, | |
| { | |
| "name": "halloween_plant_dead" | |
| }, | |
| { | |
| "name": "halloween_plant_spooky" |
| { | ||
| "name": "pumkins", | ||
| "directions": 4 | ||
| }, |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены проблемы со спрайтами и их именованием
В состоянии "pumkins" указано 4 направления (directions: 4), но:
- Найден только один файл спрайта pumkins.png
- Опечатка "pumkins" присутствует в именах файлов
- Отсутствуют спрайты для всех 4 направлений
Необходимо:
- Исправить опечатку в meta.json и переименовать файлы с "pumkins" на "pumpkins"
- Добавить недостающие спрайты для всех 4 направлений
- Убедиться, что все спрайты корректно отображаются в игре
🔗 Analysis chain
Исправьте опечатку и проверьте спрайты
- Присутствует та же опечатка в названии.
- Убедитесь, что для состояния подготовлены все 4 спрайта для разных направлений.
Примените исправление:
- "name": "pumkins",
+ "name": "pumpkins",
"directions": 4🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия всех необходимых спрайтов
find Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi -name "pumkins*.png" | wc -l
Length of output: 103
Script:
#!/bin/bash
# Проверим наличие файлов с правильным именем (pumpkins)
find Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi -name "pumpkins*.png" | wc -l
Length of output: 104
Script:
#!/bin/bash
# Посмотрим все файлы в директории .rsi для полной картины
ls -la Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/
Length of output: 1007
| id: LPPPPumpkinDeco | ||
| name: pumkin |
There was a problem hiding this comment.
Исправьте опечатку в названии тыквы
В названии сущности есть опечатка: "pumkin" должно быть "pumpkin"
Предлагаемые изменения:
id: LPPPPumpkinDeco
- name: pumkin
+ name: pumpkin📝 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.
| - type: entity | |
| id: LPPPPumpkinDeco | |
| name: pumkin | |
| - type: entity | |
| id: LPPPPumpkinDeco | |
| name: pumpkin |
|
Ожидаю изменений. |
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
⛔ Files ignored due to path filters (3)
Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumpkin_minis.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumpkin_minis2.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumpkins.pngis excluded by!**/*.png
📒 Files selected for processing (2)
- Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (2 hunks)
- Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🪛 yamllint
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml
[error] 8-8: wrong indentation: expected 4 but found 2
(indentation)
[error] 12-12: wrong indentation: expected 6 but found 4
(indentation)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 36-36: too many spaces inside brackets
(brackets)
[error] 45-45: wrong indentation: expected 4 but found 2
(indentation)
[error] 50-50: wrong indentation: expected 6 but found 4
(indentation)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 63-63: wrong indentation: expected 6 but found 4
(indentation)
[error] 67-67: wrong indentation: expected 8 but found 6
(indentation)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 79-79: wrong indentation: expected 4 but found 2
(indentation)
[error] 86-86: wrong indentation: expected 6 but found 4
(indentation)
[error] 90-90: wrong indentation: expected 8 but found 6
(indentation)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 101-101: wrong indentation: expected 4 but found 2
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 116-116: wrong indentation: expected 4 but found 2
(indentation)
[error] 123-123: wrong indentation: expected 4 but found 2
(indentation)
[error] 125-125: too many blank lines
(1 > 0) (empty-lines)
🔇 Additional comments (5)
Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json (2)
26-30: Состояния для мини-тыкв реализованы корректно!Названия состояний корректны и соответствуют общему стилю именования.
35-45: Используйте более описательные названия для растенийТекущие имена состояний (plant1, plant2, plant3, plant4) слишком общие. Рекомендуется использовать более описательные названия, отражающие тематику Хэллоуина.
Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/halloween.yml (3)
5-15: Изменения в конфигурации паука выглядят корректно!Добавление режима размещения и возможности взаимодействия улучшает функциональность декорации.
🧰 Tools
🪛 yamllint
[error] 8-8: wrong indentation: expected 4 but found 2
(indentation)
[error] 12-12: wrong indentation: expected 6 but found 4
(indentation)
[error] 13-13: too many spaces inside brackets
(brackets)
[error] 13-13: too many spaces inside brackets
(brackets)
38-71: Корректная конфигурация новой тыквы!Сущность правильно структурирована и соответствует требованиям:
- Описание на английском языке
- Корректная конфигурация спрайтов
- Правильная настройка повреждений
🧰 Tools
🪛 yamllint
[error] 45-45: wrong indentation: expected 4 but found 2
(indentation)
[error] 50-50: wrong indentation: expected 6 but found 4
(indentation)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 51-51: too many spaces inside brackets
(brackets)
[error] 63-63: wrong indentation: expected 6 but found 4
(indentation)
[error] 67-67: wrong indentation: expected 8 but found 6
(indentation)
[error] 71-71: too many spaces inside brackets
(brackets)
[error] 71-71: too many spaces inside brackets
(brackets)
96-125: 🧹 Nitpick (assertive)Улучшите описание увядших растений
Текущее описание "Withered houseplant, looks dead" довольно общее. Предлагаю сделать его более специфичным для праздника Хэллоуина.
Предлагаемое изменение:
- description: Withered houseplant, looks dead. + description: A haunting houseplant perfect for Halloween decorations, its twisted form seems to have been touched by supernatural forces.Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint
[error] 101-101: wrong indentation: expected 4 but found 2
(indentation)
[error] 109-109: wrong indentation: expected 4 but found 2
(indentation)
[error] 116-116: wrong indentation: expected 4 but found 2
(indentation)
[error] 123-123: wrong indentation: expected 4 but found 2
(indentation)
[error] 125-125: too many blank lines
(1 > 0) (empty-lines)
| "name": "pumpkins", | ||
| "directions": 4 | ||
| }, |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружено несоответствие между метаданными и файлами спрайтов
Анализ показал:
- В meta.json состояние "pumpkins" определено с 4 направлениями (directions: 4)
- Найден только один файл спрайта: pumpkins.png
- Отсутствуют отдельные файлы для каждого из 4 направлений
Необходимо:
- Добавить недостающие спрайты для всех направлений
- Или исправить значение directions в meta.json, если объект не требует разных направлений
🔗 Analysis chain
Необходима проверка спрайтов для всех направлений
Состояние "pumpkins" указывает на наличие 4 направлений (directions: 4). Необходимо убедиться, что:
- Присутствуют спрайты для всех 4 направлений
- Все спрайты корректно отображаются в игре
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия всех необходимых спрайтов для состояния pumpkins
echo "Поиск файлов спрайтов для всех направлений..."
find Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi -name "pumpkins*.png"
Length of output: 274
Script:
#!/bin/bash
# Проверяем содержимое файла meta.json для получения полной информации о состоянии
cat Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/meta.json
# Проверяем размеры и детали изображения
file Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/pumpkins.png
Length of output: 1068
| id: LPPPumpkinBigDeco | ||
| name: pumpkin | ||
| parent: BaseSign | ||
| description: Decorative pumpkins with brush marks on it | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Decoration/halloween.rsi | ||
| state: pumpkins | ||
| - type: Damageable | ||
| damageContainer: Inorganic | ||
| - type: Destructible | ||
| thresholds: | ||
| - trigger: | ||
| !type:DamageTrigger | ||
| damage: 10 | ||
| behaviors: | ||
| - !type:PlaySoundBehavior | ||
| sound: | ||
| collection: WoodDestroyHeavy | ||
| - !type:DoActsBehavior | ||
| acts: [ "Destruction" ] |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Рассмотрите возможность оптимизации конфигурации урона
Конфигурация урона и разрушения идентична для обеих тыкв. Предлагаю создать базовый прототип с общими настройками урона.
Пример реорганизации:
- type: entity
id: LPPHalloweenDestructibleBase
abstract: true
components:
- type: Damageable
damageContainer: Inorganic
- type: Destructible
thresholds:
- trigger:
!type:DamageTrigger
damage: 10
behaviors:
- !type:PlaySoundBehavior
sound:
collection: WoodDestroyHeavy
- !type:DoActsBehavior
acts: [ "Destruction" ]
- type: entity
id: LPPPumpkinBigDeco
parent: [ BaseSign, LPPHalloweenDestructibleBase ]
# ... остальные компоненты🧰 Tools
🪛 yamllint
[error] 79-79: wrong indentation: expected 4 but found 2
(indentation)
[error] 86-86: wrong indentation: expected 6 but found 4
(indentation)
[error] 90-90: wrong indentation: expected 8 but found 6
(indentation)
[error] 94-94: too many spaces inside brackets
(brackets)
[error] 94-94: too many spaces inside brackets
(brackets)
|
@Evgencheg принимай |
Описание обновления:
Медиа
Проверки
Изменения
🆑 Kest