-
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
Водолазки для Офицер Синий Щит #312
Водолазки для Офицер Синий Щит #312
Conversation
WalkthroughВ этом пулл-реквесте внесены изменения в несколько файлов, касающихся добавления новых предметов одежды и модификации конфигураций загрузок. В файл локализации добавлен новый элемент одежды с идентификатором Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Game
participant LoadoutSystem
participant Localization
participant ClothingSystem
User->>LoadoutSystem: Запросить загрузку для LPP Blue Shield Officer
LoadoutSystem->>Localization: Получить описание предмета одежды
Localization-->>LoadoutSystem: Вернуть "Водолазка офицера 'Синий Щит'"
LoadoutSystem->>ClothingSystem: Получить информацию о `LPPClothingUniformSkirtleneckBlueshield`
ClothingSystem-->>LoadoutSystem: Вернуть детали одежды
LoadoutSystem-->>User: Отобразить доступные загрузки
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 5cbb160 merging into 54a328d Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.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: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
⛔ Files ignored due to path filters (4)
Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/equipped-INNERCLOTHING.png
is excluded by!**/*.png
Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/icon.png
is excluded by!**/*.png
Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/inhand-left.png
is excluded by!**/*.png
Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/inhand-right.png
is excluded by!**/*.png
📒 Files selected for processing (5)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/misc_roles.ftl
(1 hunks)Resources/Prototypes/Loadouts/Jobs/Heads/headOfPersonnel.yml
(1 hunks)Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/misc_roles.yml
(1 hunks)Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml
(1 hunks)Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/meta.json
(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/misc_roles.ftl (1)
Pattern **/*.ftl
: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/Loadouts/Jobs/Heads/headOfPersonnel.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/misc_roles.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
🪛 yamllint
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/misc_roles.yml
[error] 37-37: wrong indentation: expected 4 but found 2
(indentation)
Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml
[error] 66-66: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (6)
Resources/Textures/_LostParadise/Clothing/Uniforms/Jumpsuit/blueshield_turtleneck.rsi/meta.json (2)
1-27
: Структура файла корректна
Метаданные содержат все необходимые поля и соответствуют стандартам RSI формата:
- Версия файла указана
- Лицензия определена
- Размеры текстур стандартные (32x32)
- Все необходимые состояния спрайта присутствуют
10-26
: Проверьте наличие всех необходимых состояний спрайта
Текущие состояния спрайта включают базовые анимации для одежды. Подтвердите, что:
- Все 4 направления для equipped-INNERCLOTHING корректно настроены
- Спрайты для левой и правой руки имеют все необходимые кадры
Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml (2)
57-66
: Конфигурация загрузки корректна!
Структура нового загрузочного набора соответствует существующим конфигурациям и правильно определяет требования к должности.
🧰 Tools
🪛 yamllint
[error] 66-66: no new line character at the end of file
(new-line-at-end-of-file)
66-66
:
Добавьте пустую строку в конец файла
Для соответствия стандартам YAML и улучшения совместимости, необходимо добавить символ новой строки в конец файла.
Примените следующее исправление:
- LPPClothingUniformSkirtleneckBlueshield
+
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint
[error] 66-66: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/misc_roles.yml (1)
35-35
: 🧹 Nitpick (assertive)
Рекомендуется улучшить описание
Текущее описание "This is the costume of a man of high rank" идентично описанию комбинезона. Предлагаю сделать его более специфичным для водолазки.
- description: This is the costume of a man of high rank.
+ description: A comfortable and professional turtleneck sweater designed for the Blueshield Officer's daily duties.
Likely invalid or redundant comment.
Resources/Prototypes/Loadouts/Jobs/Heads/headOfPersonnel.yml (1)
34-34
: Изменение корректно добавляет доступ к водолазке боцмана.
Добавление LPPBlueShieldOfficer
в список требований соответствует целям PR и позволяет офицеру получить доступ к этому предмету снаряжения.
Давайте проверим корректность ID должности:
✅ Verification successful
ID должности подтверждён и корректно используется в системе
Проверка показала, что:
- Должность
LPPBlueShieldOfficer
корректно определена в системе - ID используется в других прототипах и загрузках
- Структура соответствует стандартам проекта
Добавление этого ID в требования водолазки боцмана полностью корректно и соответствует существующей конфигурации.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверяем существование должности LPPBlueShieldOfficer
rg -l "LPPBlueShieldOfficer" "Resources/Prototypes/Jobs"
Length of output: 120
Script:
#!/bin/bash
# Search for LPPBlueShieldOfficer in all YAML files
fd -e yml -e yaml -x rg -l "LPPBlueShieldOfficer" {}
# Search for Blue Shield related job definitions
fd -e yml -e yaml -x rg -l "blueshield|blue.*shield" -i {}
Length of output: 3273
Script:
#!/bin/bash
# Проверим определение должности в файле blueshieldoficer.yml
cat "./Resources/Prototypes/_LostParadise/Roles/Jobs/CentCom/blueshieldoficer.yml"
# Проверим связанные loadouts
cat "./Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml"
Length of output: 3412
Описание обновления:
Теперь Офицер Синий Щит может получить в своем Лодауте водолазку боцмана, а также свою собственную водолзаку Офицер Синий Щит
Медиа
No
Проверки
Изменения
🆑 MrDarkSide