-
Notifications
You must be signed in to change notification settings - Fork 32
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
Фикс шаттлов утилизаторов #367
Conversation
Оказывается, при сохранении гридов, компонент SalvageShuttle не сохраняется
Обзор измененийWalkthroughВ файле YAML для карты добавлен новый объект типа Changes
Sequence DiagramsequenceDiagram
participant Map as Карта
participant Entity as SalvageShuttle
Map->>Entity: Создание объекта
Entity->>Map: Регистрация Transform
Entity->>Map: Настройка MapGrid
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
🔭 Outside diff range comments (1)
Resources/Maps/_LostParadise/Shuttles/BOX_Miner.yml (1)
Line range hint
124-124
: Содержимое Paper должно быть на английском языкеСодержимое компонента Paper должно быть на английском языке согласно правилам кодирования.
- content: >- - Ребят, пятница была АХУЕНННАЯ!! - - - И да, простите за шаттл, ИКфыжз... - - - - - - А и да, МИХАЛЫЧА НЕ ТРОЖ!!! + content: >- + Guys, Friday was AWESOME!! + + + And yeah, sorry about the shuttle, hehehe... + + + + + + Oh, and DON'T TOUCH MIKHALYCH!!!🧰 Tools
🪛 yamllint (1.35.1)
[error] 20-20: wrong indentation: expected 6 but found 4
(indentation)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
Resources/Maps/_LostParadise/Shuttles/BOX_Miner.yml
(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Maps/_LostParadise/Shuttles/BOX_Miner.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
🔇 Additional comments (3)
Resources/Maps/_LostParadise/Shuttles/BOX_Miner.yml (3)
Line range hint 1-4
: LGTM!
Структура метаданных карты корректна.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 20-20: wrong indentation: expected 6 but found 4
(indentation)
22-22
: Одобрено: Добавление компонента SalvageShuttle
Добавление компонента SalvageShuttle
решает проблему с сохранением шаттлов утилизации при сохранении сетки, как указано в целях PR.
Line range hint 1-1
: Одобрено: Расположение прототипов
Файл корректно размещен в папке _LostParadise, что соответствует требованиям к расположению прототипов.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 20-20: wrong indentation: expected 6 but found 4
(indentation)
@@ -19,6 +19,7 @@ entities: | |||
components: | |||
- type: MetaData | |||
name: NT ЭУ "Айси" | |||
- type: SalvageShuttle |
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.
Требуется перевод на английский язык
Согласно правилам кодирования, в прототипах запрещены переводы на русский язык. Замените название "NT ЭУ "Айси"" на английский вариант.
- name: NT ЭУ "Айси"
+ name: NT EU "Icy"
Committable suggestion skipped: line range outside the PR's diff.
Описание обновления:
Оказывается, при сохранении гридов, компонент SalvageShuttle не сохраняется
Медиа
Проверки
🆑 PowerKAT1