-
Notifications
You must be signed in to change notification settings - Fork 154
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
[Tweak] Pilot not idiot #1057
[Tweak] Pilot not idiot #1057
Conversation
Добавляется одежда и шкаф для пилота.
WalkthroughВ данном PR внесены многочисленные обновления, направленные на расширение и улучшение функционала экипировки пилота безопасности. Добавлены новые локализационные записи для английской и русской версий, обновлены загрузочные группы и стартовое снаряжение для роли пилота, введены новые определения сущностей для одежды, радио, шкафов и других объектов, а также обновлены метаданные текстур. Изменения затрагивают как файлы локализации, так и YAML-конфигурации, определения сущностей и метаданные ресурсов. Changes
Sequence Diagram(s)sequenceDiagram
participant Игрок
participant РольСистемы as Система Ролей
participant Загрузки as Загрузочные Конфигурации
participant Снаряжение as Система Снаряжения
Игрок->>РольСистемы: Выбор роли Pilot
РольСистемы->>Загрузки: Запрос обновлённой конфигурации
Загрузки-->>РольСистемы: Передача новых loadout-групп
РольСистемы->>Снаряжение: Назначение стартового снаряжения
Снаряжение-->>Игрок: Выдача обновлённого снаряжения (jumpsuit, шлем, перчатки, радио и т.д.)
Poem
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 (
|
RSI Diff Bot; head commit bb9423c merging into 8cba46c Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi
Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi
Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi
|
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
🧹 Nitpick comments (4)
Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl (2)
11-12
: Уточните описание фуражки!Рекомендуется конкретизировать, какие именно "признаки коммунизма" присутствуют в фуражке, например: "красная звезда" или "серп и молот".
1-13
: Исправьте пунктуацию в описаниях!Необходимо обеспечить единообразие в использовании многоточий:
- В описании комбинезона:
"...Точно не в цветах Синдиката."
- В описании фуражки:
"Фуражка пилота службы безопасности. Содержит признаки коммунизма."
Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml (2)
26-26
: Удалите завершающие пробелы.
Найдены лишние завершающие пробелы в этой строке. Пожалуйста, удалите их для соответствия стандартам форматирования.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 26-26: trailing spaces
(trailing-spaces)
49-49
: Удалите завершающие пробелы.
Пожалуйста, уберите лишние пробелы в конце строки.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 49-49: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (43)
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/icon-flash.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/off-equipped-HELMET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/off-inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/off-inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/on-equipped-HELMET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/on-inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/on-inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/equipped-HELMET-hamster.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/equipped-HELMET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/equipped-HELMET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/equipped-OUTERCLOTHING.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/equipped-FEET-vox.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/equipped-FEET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/icon-on.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/on-equipped-FEET-vox.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/on-equipped-FEET.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/on-inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/on-inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/equipped-INNERCLOTHING-monkey.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/equipped-INNERCLOTHING-reptilian.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/equipped-INNERCLOTHING.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/inhand-right.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/locked.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/sec.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/sec_open.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/sec_pilot_door.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/unlocked.png
is excluded by!**/*.png
Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/welded.png
is excluded by!**/*.png
📒 Files selected for processing (29)
Resources/Locale/en-US/_backmen/preferences/loadout-groups.ftl
(1 hunks)Resources/Locale/en-US/_backmen/prototypes/security-pilot/closet.ftl
(1 hunks)Resources/Locale/en-US/_backmen/prototypes/security-pilot/clothing.ftl
(1 hunks)Resources/Locale/en-US/_backmen/prototypes/security-pilot/radio.ftl
(1 hunks)Resources/Locale/ru-RU/_backmen/preferences/loadout-groups.ftl
(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/closet.ftl
(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl
(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/radio.ftl
(1 hunks)Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml
(1 hunks)Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml
(1 hunks)Resources/Prototypes/Corvax/Loadouts/role_loadouts.yml
(1 hunks)Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml
(1 hunks)Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/security_pilot.yml
(1 hunks)Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/supplypods.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Head/hardsuit-helmets.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Head/hats.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Head/helmets.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/OuterClothing/hardsuits.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Shoes/magboots.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Uniforms/jumpsuits.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Devices/radio.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Structures/Storage/Closets/closets.yml
(1 hunks)Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/meta.json
(1 hunks)Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/meta.json
(1 hunks)
✅ Files skipped from review due to trivial changes (11)
- Resources/Locale/en-US/_backmen/prototypes/security-pilot/closet.ftl
- Resources/Textures/Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi/meta.json
- Resources/Locale/en-US/_backmen/preferences/loadout-groups.ftl
- Resources/Textures/Backmen/Clothing/Head/Hats/secpilotcap.rsi/meta.json
- Resources/Locale/en-US/_backmen/prototypes/security-pilot/radio.ftl
- Resources/Textures/Backmen/Clothing/OuterClothing/Hardsuits/sec-pilot-hardsuit.rsi/meta.json
- Resources/Textures/Backmen/Structures/Storage/sec-pilot.rsi/meta.json
- Resources/Textures/Backmen/Clothing/Head/Hardsuits/sec-pilot-hardsuit.rsi/meta.json
- Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/radio.ftl
- Resources/Textures/Backmen/Clothing/Shoes/Boots/magboots-security.rsi/meta.json
- Resources/Textures/Backmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi/meta.json
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml
[error] 6-6: trailing spaces
(trailing-spaces)
[error] 11-11: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml
[error] 26-26: trailing spaces
(trailing-spaces)
[error] 49-49: trailing spaces
(trailing-spaces)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (28)
Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/closet.ftl (1)
1-5
: Перевод корректен!Используется правильная терминология и грамматические конструкции.
Resources/Locale/ru-RU/_backmen/preferences/loadout-groups.ftl (1)
1-4
: Переводы групп экипировки соответствуют стандартам!Корректное использование терминологии и разделение категорий.
Resources/Locale/en-US/_backmen/prototypes/security-pilot/clothing.ftl (1)
1-13
: Переводы согласованы между языками!Английская и русская версии соответствуют друг другу по смыслу и стилю. Рекомендуется исправить опечатку в идентификаторе
Jumpsui
одновременно в обоих файлах.Resources/Prototypes/Corvax/Loadouts/role_loadouts.yml (1)
22-27
: Новая конфигурация загрузочных групп для JobPilot корректна.
В группе JobPilot добавлены новые элементы:SecurityJumpsuitPilot
,SecurityOuterClothingPilot
,SecurityHeadPilot
иSecurityShoesPilot
. Убедитесь, что все новые наименования синхронизированы с соответствующими определениями в других файлах.Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml (6)
1-6
: Loadout для головного убора (Helmet) добавлен корректно.
ОпределениеSecurityPilotHelmet
с оборудованиемClothingHeadHelmetHardsuitSecurityPilot
соответствует требованиям.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 6-6: trailing spaces
(trailing-spaces)
7-11
: Loadout для кепки пилота корректен.
ИдентификаторSecurityPilotCap
и связанное оборудованиеClothingHeadCapSecurityPilot
указаны правильно.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 11-11: trailing spaces
(trailing-spaces)
12-17
: Loadout для комбинезона пилота добавлен корректно.
Проверьте, что используемое оборудованиеClothingUniformJumpsuiSecurityPilot
соответствует спецификации.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 17-17: trailing spaces
(trailing-spaces)
18-23
: Loadout для внешней одежды (CoatBomber) корректен.
Проверьте, чтоClothingOuterCoatBomber
соответствует требованиям для данного типа экипировки.
24-29
: Loadout для обуви (SecurityMagBoots) добавлен корректно.
Убедитесь, что оборудованиеClothingShoesBootsMagSecurity
задано правильно и соответствует требованиям.
30-34
: Loadout для обуви (Laceup) корректен.
Проверьте, что идентификаторLaceup
и ссылка на оборудованиеClothingShoesBootsLaceup
соответствуют спецификации.Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/security_pilot.yml (1)
1-30
: Новый объект LockerSecurityPilotFilled добавлен корректно.
Определение включает необходимые родительские типы, компонентStorageFill
и корректно заданное содержимое. Убедитесь, что все идентификаторы (например,ClothingHeadCapSecurityPilot
,ShuttleConsoleCircuitboard
) существуют в системе.Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml (2)
1-30
: Удаление определения работы Pilot.
Устаревшее определение работыPilot
было удалено. Проверьте, что новая конфигурация роли пилота задана в других соответствующих файлах и корректно интегрируется в систему.
31-47
: Обновление стартового снаряжения пилота.
Новые ссылки на экипировку (например,ClothingUniformJumpsuiSecurityPilot
,ClothingHeadCapSecurityPilot
,ClothingHandsGlovesCombat
,WeaponMultiphaseEnergyGun
иRadioSecurityPilotSupplyPod
) заданы корректно. Также проверьте, что комментарий на строке 46, указывающий на удалениеMagazinePistol
, отражает актуальные изменения.Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml (4)
16-26
: Новая loadout группа SecurityJumpsuitPilot добавлена корректно.
Группа включает элементы:SecurityPilotJumpsuit
,SecurityJumpsuit
,SecurityJumpskirt
,SeniorOfficerJumpsuit
,SeniorOfficerJumpskirt
иTrooperUniform
. Проверьте, что все указанные элементы существуют и правильно синхронизированы с остальными настройками.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 26-26: trailing spaces
(trailing-spaces)
27-36
: Новая loadout группа SecurityShoesPilot настроена корректно.
Убедитесь, что все элементы (например,JackSecBoots
,JackBoots
,SecurityWinterBoots
,SecurityMagBoots
иLaceup
) соответствуют требованиям и согласованы с другими настройками.
37-49
: Новая loadout группа SecurityHeadPilot добавлена корректно.
Проверьте, что элементы, такие какSecurityPilotHelmet
иSecurityPilotCap
, а также остальные перечисленные, соответствуют новым определениям и интегрированы в систему.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 49-49: trailing spaces
(trailing-spaces)
50-60
: Новая loadout группа SecurityOuterClothingPilot добавлена корректно.
Убедитесь, что элементы (например,ArmorVest
,ArmorVestSlim
,ClothingOuterCoatSecurityOvercoat
,ClothingOuterCoatSecurityOfficerOvercoat
,SecurityOfficerWintercoat
иCoatBomber
) заданы корректно и соответствуют требованиям.Resources/Prototypes/_Backmen/Entities/Objects/Devices/radio.yml (1)
61-79
: Новый entity для радио пилота безопасности добавлен.
ОпределениеRadioSecurityPilotSupplyPod
включает все необходимые компоненты (Sprite, Item и SpawnItemsOnUse) с корректными параметрами, а также добавлено полеsuffix
для указания специфики роли. Рекомендуется проверить, что для нового entity настроены соответствующие локализационные файлы, чтобы обеспечить единообразие перевода в игре.Resources/Prototypes/_Backmen/Entities/Structures/Storage/Closets/closets.yml (1)
56-81
: Новый entity для шкафчика пилота безопасности добавлен.
ОпределениеLockerSecurityPilot
корректно использует родительский типLockerSecurity
и задаёт все необходимые визуальные компоненты (Sprite и EntityStorageVisuals) с соответствующими слоями. Убедитесь, что в будущих изменениях добавится локализация для этого элемента, чтобы описание отображалось корректно на всех языках.Resources/Prototypes/_Backmen/Entities/Clothing/Shoes/magboots.yml (1)
56-74
: Новый entity для магнитных сапог пилота безопасности добавлен.
EntityClothingShoesBootsMagSecurity
оформлен правильно с использованием расширенного списка родительских элементов (BaseToggleClothing
иBaseSecurityContraband
), что должно обеспечить нужное поведение при экипировке. Проверьте, что визуальные элементы (спрайт и состояния) соответствуют тематике и что описание отражает специфику использования в условиях работы в невесомости.Resources/Prototypes/_Backmen/Entities/Clothing/Head/hardsuit-helmets.yml (1)
283-297
: Новый entity для шлема пилота безопасности добавлен.
EntityClothingHeadHelmetHardsuitSecurityPilot
корректно наследует отClothingHeadHelmetHardsuitSecurity
и включает необходимые компоненты, в том числе настроенный источник света (PointLight) с параметрами (color, radius, energy). Рекомендуется убедиться, что используемые спрайты соответствуют визуальному стилю других шлемов, а также что локализационные записи для данного элемента присутствуют в соответствующих файлах.Resources/Prototypes/_Backmen/Entities/Clothing/Head/hats.yml (1)
384-393
: Новый entity для кепки пилота безопасности добавлен.
EntityClothingHeadCapSecurityPilot
определён с использованием массива родительских элементов[ClothingHeadBase, BaseSecurityContraband]
, что позволяет интегрировать его в систему экипировки безопасности. Проверьте, что спрайты для компонентовSprite
иClothing
указаны корректно, а также что локализованные тексты для данного элемента добавлены в соответствующие языковые файлы для обеспечения единообразного отображения в UI.Resources/Prototypes/_Backmen/Entities/Clothing/Head/helmets.yml (1)
414-424
: Новый элемент для пилотского шлема безопасности добавлен корректно.
СущностьClothingHeadSecurityPilotHelmet
использует родительские элементы[ClothingHeadHelmetBase, BaseSecurityContraband]
, что соответствует ожидаемой функциональности. Проверьте, что пути к спрайтам (Backmen/Clothing/Head/Helmets/helmet-sec-pilot.rsi
) существуют и что указанные теги (WhitelistChameleon
иSecurityHelmet
) корректно отражают назначение шлема.Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/supplypods.yml (3)
40-48
: Маркер DropPodMarkerSecurityPilot настроен корректно.
Новый маркер наследует отDropPodMarker
, содержит корректный суффиксSecurityPilot, supply
и правильно использует компонентTimedSpawner
, ссылающийся наDropPodLaunchAnimationSecurityPilot
.
49-62
: Анимация для пилотского спуска (DropPodLaunchAnimationSecurityPilot) оформлена правильно.
Компоненты, в том числе спрайт изBackmen/Effects/droppod_falling.rsi
с указанным слоемsupplypod_falling_animation
и отступом"0,3.3"
, настроены аналогично другим анимациям. Убедитесь, что визуальные эффекты соответствуют требованиям.
63-71
: Элемент SupplyDropPodSecurityPilotHardsuit добавлен корректно.
Определение содержит нужный компонентStorageFill
, который ссылается на содержимое с идентификаторомClothingOuterHardsuitSecurityPilot
. Проверьте, что данный объект интегрирован в загрузочную конфигурацию и доступен в игре.Resources/Prototypes/_Backmen/Entities/Clothing/OuterClothing/hardsuits.yml (1)
477-493
: Новый скафандр для пилота безопасности добавлен правильно.
СущностьClothingOuterHardsuitSecurityPilot
наследует отClothingOuterHardsuitSecurity
и задаёт корректные параметры: спрайты, модификаторы скорости (walkModifier: 0.9, sprintModifier: 0.9), а также компонентHeldSpeedModifier
. Переключаемый элемент (ToggleableClothing
) ссылается наClothingHeadHelmetHardsuitSecurityPilot
. Убедитесь, что все ссылки и пути (например, для спрайтаsec-pilot-hardsuit.rsi
) корректны и доступны.Resources/Prototypes/_Backmen/Entities/Clothing/Uniforms/jumpsuits.yml (1)
895-905
: Новый jumpsuit для пилота безопасности оформлен корректно.
СущностьClothingUniformJumpsuiSecurityPilot
содержит необходимые компонентыSprite
иClothing
, ссылаясь на спрайтыBackmen/Clothing/Uniforms/jumpsuit/sec_pilot.rsi
. Описание также отражает, что данная униформа предназначена для пилота безопасности и не окрашена в цвета синдиката. Рекомендуется проверить, что добавлены соответствующие локализационные записи для данного элемента.
ent-ClothingUniformJumpsuiSecurityPilot = комбинезон пилота СБ | ||
.desc = Комбинезон специально для пилота службы безопасности. Точно не в цветах Синидката... |
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.
Исправьте опечатку в идентификаторе сущности!
В идентификаторе сущности ent-ClothingUniformJumpsuiSecurityPilot
пропущена буква 't' (должно быть Jumpsuit
).
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: 0
🧹 Nitpick comments (5)
Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml (4)
1-6
: Комментарий: Определение загрузочного набора SecurityPilotHelmetБлок для
SecurityPilotHelmet
корректно задаёт загрузочный набор с оборудованием для головы. Обратите внимание, что на строке 6 обнаружены завершающие пробелы – рекомендуется удалить их для соблюдения стиля YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 6-6: trailing spaces
(trailing-spaces)
7-11
: Комментарий: Определение загрузочного набора SecurityPilotCapКонфигурация набора для
SecurityPilotCap
выглядит корректной с указанием соответствующего элемента для головы. Проверьте, чтобы строка 11 не содержала лишних завершающих пробелов.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 11-11: trailing spaces
(trailing-spaces)
12-17
: Комментарий: Определение загрузочного набора SecurityPilotJumpsuitБлок для
SecurityPilotJumpsuit
правильно указывает загрузочный набор для комбинезона пилота. Обратите внимание на наличие завершающих пробелов в строке 17 – рекомендуется их удалить.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 17-17: trailing spaces
(trailing-spaces)
30-34
: Комментарий: Определение загрузочного набора LaceupБлок для
Laceup
выглядит правильно, оборудование для обуви указано корректно. Рекомендуется проверить, что форматирование (отсутствие завершающих пробелов) соответствует стандартам.Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml (1)
31-47
: Комментарий: Обновление начального снаряжения (PilotGear)Блок
PilotGear
обновлён для корректной экипировки:
- Комбинезон заменён на
ClothingUniformJumpsuitSecurityPilot
.- Головной элемент изменён на
ClothingHeadCapSecurityPilot
.- Добавлены перчатки (
ClothingHandsGlovesCombat
) и изменён состав предметов в карманах (первый карман –WeaponMultiphaseEnergyGun
, второй –RadioSecurityPilotSupplyPod
).Также обратите внимание на закомментированную строку на строке 46; если она больше не нужна, её можно удалить, либо добавить пояснение о её предназначении.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
Resources/Locale/en-US/_backmen/prototypes/security-pilot/clothing.ftl
(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl
(1 hunks)Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml
(1 hunks)Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml
(1 hunks)Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/security_pilot.yml
(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Uniforms/jumpsuits.yml
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (4)
- Resources/Prototypes/_Backmen/Catalog/Fills/Lockers/security_pilot.yml
- Resources/Prototypes/_Backmen/Entities/Clothing/Uniforms/jumpsuits.yml
- Resources/Locale/en-US/_backmen/prototypes/security-pilot/clothing.ftl
- Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml
[error] 6-6: trailing spaces
(trailing-spaces)
[error] 11-11: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (3)
Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml (2)
18-23
: Комментарий: Определение загрузочного набора CoatBomberНабор для
CoatBomber
определён корректно с указанием оборудования для внешней одежды. Убедитесь, что имяClothingOuterCoatBomber
соответствует требуемой номенклатуре.
24-29
: Комментарий: Определение загрузочного набора SecurityMagBootsКонфигурация набора для
SecurityMagBoots
корректна – указана обувь с правильным идентификатором.Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml (1)
1-30
: Комментарий: Удаление определения работы PilotУдаление определения работы
Pilot
произведено корректно, если это соответствует новой архитектуре ролей. Убедитесь, что все ссылки на данную работу были обновлены или удалены в других частях проекта.
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.
Спидмердж
Описание PR
Данный PR делает пилота СБ не бесполезным...
Тип PR
🆑
Summary by CodeRabbit