Новогодние штучки #364
Новогодние штучки #364
Conversation
BL02DL
added
Изменения: Локализация
ОбзорИзменения представляют собой добавление новых локализационных файлов и ресурсов для праздничных (новогодних) предметов в игре. Включены новые сущности для украшений, такие как носки, колокольчики, гирлянды и конфеты, с локализацией на английском и русском языках, а также соответствующие текстуры и метаданные. Также расширена функциональность подарков с новыми компонентами и вариантами спавна предметов. Изменения
Диаграмма последовательностиsequenceDiagram
participant Player
participant Present
participant ItemSpawner
Player->>Present: Открывает подарок
Present->>ItemSpawner: Запрос на спавн случайного предмета
ItemSpawner-->>Present: Выбирает и спавнит предмет
Present-->>Player: Выдаёт случайный предмет
Диаграмма демонстрирует механику открытия подарка с случайным содержимым, где при взаимодействии игрока с подарком происходит генерация и выдача случайного предмета. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (4)
🧰 Additional context used📓 Path-based instructions (4)Resources/Prototypes/Entities/Objects/Decoration/present.yml (1)Pattern Все добавляемые прототипы должны быть в нашей папке. Никогда не предлагать ничего по отступам При удалении дополнительного дополнения по типу Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (1)Pattern А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (1)Pattern А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml (1)Pattern Все добавляемые прототипы должны быть в нашей папке. Никогда не предлагать ничего по отступам При удалении дополнительного дополнения по типу 🪛 yamllint (1.35.1)Resources/Prototypes/Entities/Objects/Decoration/present.yml[error] 10-10: wrong indentation: expected at least 5 (indentation) Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml[error] 7-7: wrong indentation: expected at least 3 (indentation) [error] 18-18: wrong indentation: expected at least 3 (indentation) [error] 29-29: wrong indentation: expected at least 3 (indentation) [error] 40-40: wrong indentation: expected at least 3 (indentation) [error] 51-51: wrong indentation: expected at least 3 (indentation) [error] 58-58: wrong indentation: expected at least 5 (indentation) [error] 64-64: wrong indentation: expected 10 but found 8 (indentation) [error] 80-80: wrong indentation: expected 4 but found 2 (indentation) [error] 96-96: wrong indentation: expected 4 but found 2 (indentation) [error] 99-99: wrong indentation: expected 6 but found 4 (indentation) [error] 100-100: too many spaces inside brackets (brackets) [error] 100-100: too many spaces inside brackets (brackets) [warning] 125-125: too few spaces before comment (comments) [warning] 125-125: missing starting space in comment (comments) [warning] 159-159: too few spaces before comment (comments) [warning] 161-161: too few spaces before comment (comments) [warning] 163-163: too few spaces before comment (comments) [warning] 439-439: too few spaces before comment (comments) [warning] 441-441: too few spaces before comment (comments) 🔇 Additional comments (13)Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (3)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (6)
Resources/Prototypes/Entities/Objects/Decoration/present.yml (1)
🧰 Tools🪛 yamllint (1.35.1)[error] 10-10: wrong indentation: expected at least 5 (indentation) Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml (3)
🧰 Tools🪛 yamllint (1.35.1)[error] 7-7: wrong indentation: expected at least 3 (indentation) [error] 18-18: wrong indentation: expected at least 3 (indentation) [error] 29-29: wrong indentation: expected at least 3 (indentation)
🧰 Tools🪛 yamllint (1.35.1)[error] 40-40: wrong indentation: expected at least 3 (indentation)
🧰 Tools🪛 yamllint (1.35.1)[error] 96-96: wrong indentation: expected 4 but found 2 (indentation) [error] 99-99: wrong indentation: expected 6 but found 4 (indentation) [error] 100-100: too many spaces inside brackets (brackets) [error] 100-100: too many spaces inside brackets (brackets) [warning] 125-125: too few spaces before comment (comments) [warning] 125-125: missing starting space in comment (comments) [warning] 159-159: too few spaces before comment (comments) [warning] 161-161: too few spaces before comment (comments) [warning] 163-163: too few spaces before comment (comments) [warning] 439-439: too few spaces before comment (comments) [warning] 441-441: too few spaces before comment (comments) 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 fd8a955 merging into 6504f06 Resources/Textures/Objects/Decoration/present.rsi
Resources/Textures/_LostParadise/Objects/Consumable/candystick.rsi
Resources/Textures/_LostParadise/Structures/Wallmounts/christmasbell.rsi
Resources/Textures/_LostParadise/Structures/Wallmounts/christmaslights.rsi
Resources/Textures/_LostParadise/Structures/Wallmounts/christmassock.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: 6
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (12)
Resources/Textures/Objects/Decoration/present.rsi/blue.pngis excluded by!**/*.pngResources/Textures/Objects/Decoration/present.rsi/green.pngis excluded by!**/*.pngResources/Textures/Objects/Decoration/present.rsi/purple.pngis excluded by!**/*.pngResources/Textures/Objects/Decoration/present.rsi/red.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Consumable/candystick.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Consumable/candystick.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Objects/Consumable/candystick.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Structures/Wallmounts/christmasbell.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Structures/Wallmounts/christmaslights.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Structures/Wallmounts/christmassock.rsi/green.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Structures/Wallmounts/christmassock.rsi/orange.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Structures/Wallmounts/christmassock.rsi/red.pngis excluded by!**/*.png
📒 Files selected for processing (11)
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl(1 hunks)Resources/Prototypes/Entities/Objects/Decoration/present.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml(2 hunks)Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml(1 hunks)Resources/Textures/Objects/Decoration/present.rsi/meta.json(2 hunks)Resources/Textures/_LostParadise/Objects/Consumable/candystick.rsi/meta.json(1 hunks)Resources/Textures/_LostParadise/Structures/Wallmounts/christmasbell.rsi/meta.json(1 hunks)Resources/Textures/_LostParadise/Structures/Wallmounts/christmaslights.rsi/meta.json(1 hunks)Resources/Textures/_LostParadise/Structures/Wallmounts/christmassock.rsi/meta.json(1 hunks)
🧰 Additional context used
📓 Path-based instructions (6)
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/Entities/Objects/Decoration/present.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/structures/wallmounts/newyear.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🪛 yamllint (1.35.1)
Resources/Prototypes/Entities/Objects/Decoration/present.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)
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 18-18: wrong indentation: expected at least 3
(indentation)
[error] 29-29: wrong indentation: expected at least 3
(indentation)
[error] 40-40: wrong indentation: expected at least 3
(indentation)
[error] 51-51: wrong indentation: expected at least 3
(indentation)
[error] 58-58: wrong indentation: expected at least 5
(indentation)
[error] 64-64: wrong indentation: expected 10 but found 8
(indentation)
[error] 80-80: wrong indentation: expected 4 but found 2
(indentation)
🔇 Additional comments (13)
Resources/Textures/_LostParadise/Structures/Wallmounts/christmasbell.rsi/meta.json (1)
1-14: Файл метаданных выглядит правильно.
Параметры (version, license, size, states) корректны, проблем не обнаружено.
Resources/Textures/_LostParadise/Structures/Wallmounts/christmaslights.rsi/meta.json (1)
1-15: Файл метаданных корректен.
Значения полей (version, license, size) и указанные состояния соответствуют ожидаемому формату.
Resources/Textures/Objects/Decoration/present.rsi/meta.json (2)
4-4: Корректное обновление информации об авторских правах
Правильно добавлена атрибуция для новых цветовых вариантов (синий, фиолетовый, зеленый).
✅ Verification successful
Подтверждено наличие всех новых текстур и корректность авторских прав
Проверка подтвердила наличие всех заявленных текстур для новых цветовых вариантов:
blue.pngpurple.pnggreen.png
Информация об авторских правах в meta.json корректно отражает вклад обоих авторов.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия файлов текстур для новых цветов
fd -e png "blue|purple|green" "Resources/Textures/Objects/Decoration/present.rsi/"
Length of output: 262
15-26: Новые цветовые состояния добавлены корректно
Структура новых состояний соответствует существующему формату. Рекомендуется убедиться, что все текстуры для новых цветов присутствуют в директории.
✅ Verification successful
Текстуры для новых состояний подтверждены
Проверка показала, что все необходимые файлы текстур (red.png, blue.png, purple.png, green.png) присутствуют в директории present.rsi. Хотя прямые ссылки на эти состояния в прототипах не найдены, наличие соответствующих текстур подтверждает корректность добавления новых цветовых состояний.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка использования новых состояний в прототипах сущностей
rg -l "present.(red|blue|purple|green)" "Resources/Prototypes"
Length of output: 62
Script:
#!/bin/bash
# Проверяем наличие файлов текстур для новых цветов
fd -e png "red|blue|purple|green" "Resources/Textures/Objects/Decoration/present.rsi"
# Проверяем использование состояний в прототипах с другим форматом поиска
rg -l "sprite-state = \"(red|blue|purple|green)\"" "Resources/Prototypes"
Length of output: 395
Resources/Prototypes/_LostParadise/Entities/Objects/Consumable/food/halloween.yml (2)
15-18: Форматирование выглядит корректно!
Структура RandomSprite компонента правильная, и все состояния спрайтов определены корректно.
36-36: Тег определен правильно!
Использование тега LPPCandy согласуется с новогодними конфетами, что обеспечивает правильную категоризацию.
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/halloween.yml (1)
27-27: Описание соответствует требованиям!
Текст описания на английском языке, что соответствует требованиям проекта.
Resources/Prototypes/_LostParadise/Entities/Structures/Wallmounts/newyear.yml (2)
74-87: Проверьте цвет гирлянды
Гирлянда использует яркий красный цвет (#D6001C), что может быть слишком интенсивным. Рекомендуется проверить это значение в игре.
Также убедитесь, что спрайт christmaslights.rsi существует:
🧰 Tools
🪛 yamllint (1.35.1)
[error] 80-80: wrong indentation: expected 4 but found 2
(indentation)
45-73: Проверьте консистентность с Halloween конфетами
Новогодняя конфета (LPPCandyNewYear) использует другой спрайт и структуру компонентов, отличную от хэллоуинских конфет. Убедитесь, что это намеренное изменение.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 51-51: wrong indentation: expected at least 3
(indentation)
[error] 58-58: wrong indentation: expected at least 5
(indentation)
[error] 64-64: wrong indentation: expected 10 but found 8
(indentation)
Resources/Prototypes/Entities/Objects/Decoration/present.yml (4)
10-18: Отличная реализация случайных цветов подарков!
Добавление различных цветовых вариантов (красный, синий, фиолетовый, зеленый) хорошо подходит для праздничной тематики. Корректно реализовано через компонент RandomSprite с правильным маппингом состояний.
🧰 Tools
🪛 yamllint (1.35.1)
[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)
Line range hint 47-289: Необходимо проверить баланс пула подарков
В списке подарков есть потенциально проблемные предметы:
- Оружие (Machete, CombatKnife, Spear)
- Утилитарные предметы с высокой ценностью (ClothingBeltUtilityFilled)
Рекомендации:
- Добавить вероятности (prob) для более редких предметов
- Возможно, следует вынести опасные предметы в отдельный пул подарков
Хотите, чтобы я предложил структуру вероятностей для различных категорий подарков?
🧰 Tools
🪛 yamllint (1.35.1)
[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)
Line range hint 341-386: Хорошо сбалансированные вероятности денежных призов!
Грамотно реализована система вероятностей для различных сумм:
- Маленькие суммы (10-100) имеют высокую вероятность
- Средние суммы (500-1000) умеренно редкие
- Большие суммы (5000+) соответствующе редкие
- Джекпот (1000000) имеет подходящую вероятность 0.0001%
🧰 Tools
🪛 yamllint (1.35.1)
[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)
Line range hint 388-407: Корректная реализация мусора от подарков!
Сущность правильно настроена:
- Имеет компонент SpaceGarbage
- Помечена тегом Trash
- Хорошее описание, подчеркивающее игровой процесс
🧰 Tools
🪛 yamllint (1.35.1)
[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)
| ent-LPPNewYearSocksRed = socks | ||
| .desc = hanging socks on the wall, like you should put gifts there... | ||
| ent-LPPNewYearSocksGreen = socks | ||
| .desc = hanging socks on the wall, like you should put gifts there... | ||
| ent-LPPNewYearSocksOrange = socks | ||
| .desc = hanging socks on the wall, like you should put gifts there... | ||
| ent-LPPNewYearBell = new year bell | ||
| .desc = New Year's bell with a wreath, festive mood.. | ||
| ent-LPPCandyNewYear = candy | ||
| .desc = A New Year's candy that feels like the end of the past. | ||
| ent-LPPLightbulbsNewYear = light bulbs | ||
| .desc = Hanging garland, not the best source of light |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Необходимо скорректировать регистр согласно требованиям к переводам, начинающимся с ent-.
Согласно вашим руководствам, все переводы, ключ которых начинается с ent-, должны идти с заглавной буквы после знака «=», включая описание .desc. Например:
-ent-LPPNewYearSocksRed = socks
- .desc = hanging socks on the wall, like you should put gifts there...
+ent-LPPNewYearSocksRed = Socks
+ .desc = Hanging socks on the wall, like you should put gifts there...Аналогично нужно применить к остальным строкам (стр. 1-12).
Committable suggestion skipped: line range outside the PR's diff.
| ent-LPPNewYearSocksRed = носок | ||
| .desc = Новогодний носок, как будто туда надо класть подарки... | ||
| ent-LPPNewYearSocksGreen = носок | ||
| .desc = Новогодний носок, как будто туда надо класть подарки... | ||
| ent-LPPNewYearSocksOrange = носок | ||
| .desc = Новогодний носок, как будто туда надо класть подарки... | ||
| ent-LPPNewYearBell = new year bell | ||
| .desc = Новогодний колокольчик с венком, праздничное настроение.. | ||
| ent-LPPCandyNewYear = конфетка | ||
| .desc = Новогодняя конфета, которая напоминает о конце прошлого. | ||
| ent-LPPLightbulbsNewYear = гирлянда | ||
| .desc = Новогодняя гирлянда озаряющая красками нового года. |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Следует привести текст к заглавной букве после «=» для ключей, начинающихся с ent-.
По вашим гайдлайнам переводы вида ent-... должны быть с заглавной буквы. Также описания .desc должны начинаться с заглавной. Например:
-ent-LPPNewYearSocksRed = носок
- .desc = Новогодний носок, как будто туда надо класть подарки...
+ent-LPPNewYearSocksRed = Носок
+ .desc = Новогодний носок, как будто туда надо класть подарки...Committable suggestion skipped: line range outside the PR's diff.
| "states": [ | ||
| { | ||
| "name": "red" | ||
| }, | ||
| { | ||
| "name": "orange" | ||
| }, | ||
| { | ||
| "name": "green" | ||
| } | ||
| ] |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Рекомендуется добавить дополнительные свойства состояний
Текущие определения состояний содержат только имена. Рассмотрите возможность добавления следующих свойств для улучшения функциональности:
directions: для поддержки разных направлений размещенияdelays: если планируется анимация
"states": [
{
"name": "red"
+ "directions": 1,
+ "delays": [[1.0]]
},
{
"name": "orange"
+ "directions": 1,
+ "delays": [[1.0]]
},
{
"name": "green"
+ "directions": 1,
+ "delays": [[1.0]]
}
]📝 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.
| "states": [ | |
| { | |
| "name": "red" | |
| }, | |
| { | |
| "name": "orange" | |
| }, | |
| { | |
| "name": "green" | |
| } | |
| ] | |
| "states": [ | |
| { | |
| "name": "red", | |
| "directions": 1, | |
| "delays": [[1.0]] | |
| }, | |
| { | |
| "name": "orange", | |
| "directions": 1, | |
| "delays": [[1.0]] | |
| }, | |
| { | |
| "name": "green", | |
| "directions": 1, | |
| "delays": [[1.0]] | |
| } | |
| ] |
| "states": [ | ||
| { | ||
| "name": "icon" | ||
| }, | ||
| { | ||
| "name": "inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "inhand-right", | ||
| "directions": 4 | ||
| } | ||
| ] |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Структура файла корректна
Правильно определены состояния для иконки и спрайтов в руках с 4 направлениями. Если в будущем планируется добавить анимацию (например, мерцание), можно будет добавить свойство delays.
"states": [
{
"name": "icon"
+ "delays": [[1.0]]
},
{
"name": "inhand-left",
"directions": 4
+ "delays": [[1.0], [1.0], [1.0], [1.0]]
},
{
"name": "inhand-right",
"directions": 4
+ "delays": [[1.0], [1.0], [1.0], [1.0]]
}
]Committable suggestion skipped: line range outside the PR's diff.
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksRed | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: red | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksGreen | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: green | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksOrange | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: orange | ||
| noRot: false | ||
|
|
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Рекомендуется рефакторинг сущностей носков
Три сущности носков (Red, Green, Orange) содержат дублирующийся код.
Предлагаю создать базовый прототип для носков:
- type: entity
parent: BaseSign
id: LPPNewYearSocksBase
abstract: true
name: socks
description: hanging socks on the wall, like you should put gifts there...
components:
- type: Sprite
sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi
noRot: falseИ затем использовать его как родителя:
- type: entity
parent: LPPNewYearSocksBase
id: LPPNewYearSocksRed
components:
- type: Sprite
state: red
- type: entity
parent: LPPNewYearSocksBase
id: LPPNewYearSocksGreen
components:
- type: Sprite
state: green
- type: entity
parent: LPPNewYearSocksBase
id: LPPNewYearSocksOrange
components:
- type: Sprite
state: orange🧰 Tools
🪛 yamllint (1.35.1)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 18-18: wrong indentation: expected at least 3
(indentation)
[error] 29-29: wrong indentation: expected at least 3
(indentation)
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksRed | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: red | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksGreen | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: green | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearSocksOrange | ||
| name: socks | ||
| description: hanging socks on the wall, like you should put gifts there... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmassock.rsi | ||
| state: orange | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPNewYearBell | ||
| name: new year bell | ||
| description: New Year's bell with a wreath, festive mood.. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmasbell.rsi | ||
| state: icon | ||
| noRot: false | ||
|
|
||
| - type: entity | ||
| name: candy | ||
| parent: FoodBase | ||
| id: LPPCandyNewYear | ||
| description: A New Year's candy that feels like the end of the past. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Consumable/candystick.rsi | ||
| state: icon | ||
| - type: Item | ||
| sprite: _LostParadise/Objects/Consumable/candystick.rsi | ||
| size: Tiny | ||
| shape: | ||
| - 0, 0, 0, 0 | ||
| - type: SolutionContainerManager | ||
| solutions: | ||
| food: | ||
| maxVol: 5 | ||
| reagents: | ||
| - ReagentId: Sugar | ||
| Quantity: 3 | ||
| - ReagentId: Nutriment | ||
| Quantity: 1 | ||
| - ReagentId: Theobromine | ||
| Quantity: 1 | ||
| - type: Tag | ||
| tags: | ||
| - LPPCandy | ||
|
|
||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPLightbulbsNewYear | ||
| name: light bulbs | ||
| description: Hanging garland, not the best source of light | ||
| components: | ||
| - type: PointLight | ||
| radius: 2.0 | ||
| energy: 2.5 | ||
| color: "#D6001C" | ||
| - type: Sprite | ||
| sprite: _LostParadise/Structures/Wallmounts/christmaslights.rsi | ||
| state: icon | ||
| noRot: false |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Исправьте отступы в файле
Статический анализатор yamllint обнаружил несколько проблем с отступами. Рекомендуется использовать консистентные отступы во всем файле.
Основные места для исправления:
- Компоненты должны иметь отступ 4 пробела
- Вложенные свойства должны иметь дополнительный отступ 2 пробела
- Массивы должны иметь отступ на уровень больше родительского элемента
🧰 Tools
🪛 yamllint (1.35.1)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 18-18: wrong indentation: expected at least 3
(indentation)
[error] 29-29: wrong indentation: expected at least 3
(indentation)
[error] 40-40: wrong indentation: expected at least 3
(indentation)
[error] 51-51: wrong indentation: expected at least 3
(indentation)
[error] 58-58: wrong indentation: expected at least 5
(indentation)
[error] 64-64: wrong indentation: expected 10 but found 8
(indentation)
[error] 80-80: wrong indentation: expected 4 but found 2
(indentation)
Описание обновления: Добавление новогодних украшений
Медиа
Проверки
Изменения
🆑 BL02DL