переделки?) #274
переделки?) #274
Conversation
WalkthroughВ данном пулл-запросе внесены изменения в несколько файлов, касающиеся удаления функциональности, связанной с пожертвованиями в интерфейсе редактирования профиля гуманоидов. Убраны элементы пользовательского интерфейса и логика, связанные с "Lost Paradise Donate Preferences". Также удалены классы и свойства, связанные с пожертвованиями, из модели базы данных, что упрощает структуру кода. Остальные компоненты и их связи остались неизменными. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UI
participant ProfileManager
participant RequirementsSystem
User->>UI: Открыть редактор профиля
UI->>ProfileManager: Запросить данные профиля
ProfileManager-->>UI: Вернуть данные профиля
UI->>User: Отобразить данные профиля
User->>UI: Изменить профиль
UI->>ProfileManager: Сохранить изменения
ProfileManager->>RequirementsSystem: Проверить требования
RequirementsSystem-->>ProfileManager: Вернуть результат проверки
ProfileManager-->>UI: Обновить интерфейс
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
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
Статус: Требуется одобрение
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 8
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (9)
- Content.Client/Preferences/UI/HumanoidProfileEditor.xaml (0 hunks)
- Content.Client/Preferences/UI/HumanoidProfileEditor.xaml.cs (0 hunks)
- Content.IntegrationTests/Tests/Preferences/ServerDbSqliteTests.cs (0 hunks)
- Content.Shared/Customization/Systems/CharacterRequirements.Logic.cs (2 hunks)
- Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2 hunks)
- Content.Shared/Customization/Systems/CharacterRequirementsSystem.cs (2 hunks)
- Content.Shared/Preferences/HumanoidCharacterProfile.cs (1 hunks)
- Resources/Locale/en-US/_LostParadise/loadouts.ftl (1 hunks)
- Resources/Locale/ru-RU/_LostParadise/loadouts.ftl (1 hunks)
💤 Files with no reviewable changes (3)
- Content.Client/Preferences/UI/HumanoidProfileEditor.xaml
- Content.Client/Preferences/UI/HumanoidProfileEditor.xaml.cs
- Content.IntegrationTests/Tests/Preferences/ServerDbSqliteTests.cs
🧰 Additional context used
📓 Path-based instructions (2)
Resources/Locale/en-US/_LostParadise/loadouts.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/_LostParadise/loadouts.ftl (1)
Pattern
**/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текстВсе переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🔇 Additional comments (3)
Resources/Locale/ru-RU/_LostParadise/loadouts.ftl (1)
1-1: Отлично реализовано!Новая запись для категории загрузки "Спонсорские вещи" корректно добавлена в файл локализации. Формат и содержание соответствуют требованиям FTL и правилам локализации.
Content.Shared/Customization/Systems/CharacterRequirementsSystem.cs (1)
31-32: Одобрено: Добавление проверки спонсорских требований.Добавление
CharacterSponsorRequirementв список проверяемых требований соответствует целям PR по внедрению функциональности для спонсорских предметов. Изменение логично интегрировано в существующую структуру кода.Content.Shared/Customization/Systems/CharacterRequirements.Logic.cs (1)
100-101: Добавлено условие для проверки CharacterSponsorRequirement.Новое условие расширяет список требований, которые проверяются особым образом. Это изменение, по-видимому, связано с добавлением функциональности, относящейся к спонсорам.
Убедитесь, что:
- Логика обработки CharacterSponsorRequirement соответствует общей архитектуре системы требований.
- Добавлены соответствующие тесты для проверки нового условия.
- Документация класса обновлена с учетом новой функциональности.
Для проверки использования CharacterSponsorRequirement выполните следующий скрипт:
✅ Verification successful
Проверка использования CharacterSponsorRequirement подтверждена.
Класс
CharacterSponsorRequirementкорректно определен и используется в различных частях кодовой базы. Изменения, связанные с добавлением этого условия, соответствуют общей архитектуре системы требований.Убедитесь, что:
- Логика обработки
CharacterSponsorRequirementсогласована с остальной частью системы.- Добавлены соответствующие тесты для нового условия.
- Документация обновлена с учетом новой функциональности.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Поиск использования CharacterSponsorRequirement в кодовой базе echo "Поиск определения CharacterSponsorRequirement:" ast-grep --lang csharp --pattern 'class CharacterSponsorRequirement : $_' echo "\nПоиск использования CharacterSponsorRequirement:" rg --type csharp "CharacterSponsorRequirement"Length of output: 1334
Content.Shared/Customization/Systems/CharacterRequirementsSystem.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Logic.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
github-actions
bot
added
the
СВЯТЫЕ УГОДНИКИ
There was a problem hiding this comment.
Actionable comments posted: 6
🧹 Outside diff range comments (1)
Content.Server.Database/Migrations/Postgres/PostgresServerDbContextModelSnapshot.cs (1)
Обнаружены оставшиеся упоминания
Donateв кодовой базе.Удаление сущности
Donateне было полностью завершено. В следующих файлах найдены упоминанияDonate:
Файлы миграций:
./Content.Server.Database/Migrations/Sqlite/20240906152019_Donate.Designer.cs./Content.Server.Database/Migrations/Postgres/20240906152028_Donate.cs./Content.Server.Database/Migrations/Postgres/20240906152028_Donate.Designer.cs./Content.Server.Database/Migrations/Sqlite/20240906152019_Donate.cs- и другие аналогичные файлы.
Кодовые файлы:
- Упоминания
Donateв нескольких файлах миграций, таких как1984 Backpack Prefs.Designer.cs,FixShitCode.Designer.cs, иCustomSpecieName.Designer.cs.Рекомендуемые действия:
Удаление файлов миграций:
- Полностью удалить все файлы миграций, связанные с
Donate, чтобы избежать конфликтов и дублирования.Очистка кода:
- Удалить все упоминания
Donateиз кодовых файлов, включая свойства, индексы и навигационные свойства.Создание новых миграций:
- Сгенерировать и применить новые миграции для корректного обновления схемы базы данных без сущности
Donate.Тестирование:
- Провести тщательное тестирование приложения, чтобы убедиться, что удаление
Donateне привело к непредвиденным проблемам или ошибкам в других частях системы.Обновление документации:
- При необходимости обновить документацию и комментарии в коде, связанные с удаленной сущностью.
🔗 Analysis chain
Line range hint
1-1830: Изменения в модели данных одобрены.Удаление сущности
Donateи связанных с ней свойств соответствует целям PR по удалению старой вкладки для личных предметов. Эти изменения корректны и согласуются с описанием PR.Рекомендуется выполнить следующие дополнительные шаги:
- Убедитесь, что созданы соответствующие миграции для обновления схемы базы данных.
- Проверьте, не требуется ли дополнительная обработка существующих данных, связанных с удаленной сущностью
Donate.- Обновите все запросы и операции в коде, которые могли использовать удаленную сущность
Donate.- Проведите тщательное тестирование для подтверждения, что удаление этой сущности не вызвало непредвиденных побочных эффектов в других частях приложения.
Для проверки наличия миграций и связанных изменений, выполните следующий скрипт:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверка наличия файлов миграций echo "Поиск файлов миграций:" fd --type f --glob "*Donate*.cs" --exec echo "Найден файл миграции: {}" # Поиск использования Donate в коде echo "\nПоиск оставшихся упоминаний Donate в коде:" rg --type csharp "Donate"Length of output: 5537
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (8)
- Content.Server.Database/Migrations/Postgres/20241014210800_Delete.Designer.cs (1 hunks)
- Content.Server.Database/Migrations/Postgres/20241014210800_Delete.cs (1 hunks)
- Content.Server.Database/Migrations/Postgres/PostgresServerDbContextModelSnapshot.cs (1 hunks)
- Content.Server.Database/Migrations/Sqlite/20241014210750_Delete.Designer.cs (1 hunks)
- Content.Server.Database/Migrations/Sqlite/20241014210750_Delete.cs (1 hunks)
- Content.Server.Database/Migrations/Sqlite/SqliteServerDbContextModelSnapshot.cs (1 hunks)
- Content.Server.Database/Model.cs (0 hunks)
- Content.Server/Database/ServerDbBase.cs (0 hunks)
💤 Files with no reviewable changes (2)
- Content.Server.Database/Model.cs
- Content.Server/Database/ServerDbBase.cs
🧰 Additional context used
🔇 Additional comments (7)
Content.Server.Database/Migrations/Sqlite/20241014210750_Delete.cs (1)
8-9: Уточните назначение миграцииНазвание класса "Delete" предполагает, что эта миграция предназначена для удаления чего-то из базы данных. Однако, без реализации методов
UpиDown, назначение этой миграции неясно.Пожалуйста, уточните цель этой миграции и добавьте соответствующие комментарии к классу, объясняющие его назначение.
Content.Server.Database/Migrations/Postgres/20241014210800_Delete.cs (1)
8-8: Уточните назначение класса "Delete"Название класса "Delete" предполагает, что эта миграция должна что-то удалять, но методы пусты.
Пожалуйста, уточните назначение этой миграции. Если удаление не требуется, рассмотрите возможность переименования класса или удаления файла.
Content.Server.Database/Migrations/Sqlite/SqliteServerDbContextModelSnapshot.cs (3)
Line range hint
1486-1516: Уточнение по сущностиSponsorВ снимке базы данных присутствует сущность
Sponsor, которая компилируется условно на основе флагаLPP_Sponsors. Эта сущность может быть связана с новой вкладкой для спонсорских предметов, упомянутой в целях PR.Пожалуйста, уточните:
- Является ли сущность
Sponsorновой или измененной в рамках этого PR?- Как эта сущность связана с новой вкладкой для спонсорских предметов?
- Планируется ли использование этой сущности для хранения информации о личных предметах?
Для проверки истории изменений этой сущности можно использовать следующий скрипт:
#!/bin/bash # Проверка истории изменений сущности Sponsor git log -p -- Content.Server.Database/Migrations/Sqlite/SqliteServerDbContextModelSnapshot.cs | grep -A 20 "Sponsor"
Line range hint
1-1759: Итоговый комментарий по изменениям в файлеИзменения в файле
SqliteServerDbContextModelSnapshot.csв целом соответствуют целям PR:
- Удаление сущности
Donateотражает удаление старой вкладки для личных предметов.- Наличие сущности
Sponsorможет быть связано с добавлением новой вкладки для спонсорских предметов, но требуется дополнительное уточнение.Для полного соответствия целям PR рекомендуется:
- Подтвердить, что удаление сущности
Donateне вызвало нежелательных побочных эффектов в других частях приложения.- Уточнить роль сущности
Sponsorв контексте новой вкладки для спонсорских предметов.- Проверить, что все необходимые миграции базы данных были созданы для отражения этих изменений.
Выполните следующий скрипт для проверки наличия соответствующих миграций:
#!/bin/bash # Поиск миграций, связанных с изменениями в Donate и Sponsor rg -i '(donate|sponsor)' Content.Server.Database/Migrations/
Line range hint
1-1759: Удаление сущностиDonateвыполнено корректно.Сущность
Donateбыла полностью удалена из снимка базы данных, что соответствует целям PR по удалению старой вкладки для личных предметов. Изменение выглядит правильным в контексте этого файла.Рекомендуется проверить другие части кодовой базы на наличие оставшихся ссылок на удаленную сущность
Donate. Выполните следующий скрипт для поиска потенциальных ссылок:Content.Server.Database/Migrations/Sqlite/20241014210750_Delete.Designer.cs (1)
1155-1197:⚠️ Potential issueПроверьте удаление таблицы
Sponsorиз базы данныхЦель данного PR — удалить функциональность, связанную со спонсорством. Однако в миграции
DeleteтаблицаSponsorвсе еще определяется внутри директив#if LPP_Sponsors(строки 1155-1197). Пожалуйста, убедитесь, что таблицаSponsorполностью удалена из схемы базы данных, если она больше не используется.Чтобы убедиться, что класс
Sponsorи связанные с ним элементы больше не используются в кодовой базе, выполните следующий скрипт:✅ Verification successful
Необходимо проверить, используется ли директива
LPP_Sponsorsв кодовой базе. Пожалуйста, выполните следующий скрипт:
Пожалуйста, выполните следующий скрипт для поиска любых упоминаний директивы
LPP_Sponsorsв кодовой базе:
Пожалуйста, выполните следующий скрипт для поиска определений директивы
LPP_Sponsorsв файлах конфигурации сборки и других связанных файлах:
Удаление таблицы
Sponsorподтверждено и полностью удалено из схемы базы данных.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Поиск использования класса Sponsor в кодовой базе # Ожидается: Нет результатов, если класс больше не используется rg --type cs 'Sponsor'Length of output: 26152
Script:
#!/bin/bash # Описание: Поиск определения директивы LPP_Sponsors в кодовой базе # Ожидается: Нет результатов, если директива не используется rg --type cs '#define\s+LPP_Sponsors'Length of output: 39
Script:
#!/bin/bash # Описание: Поиск всех упоминаний директивы LPP_Sponsors в кодовой базе # Ожидается: Нет результатов, если директива не используется нигде rg 'LPP_Sponsors'Length of output: 12857
Script:
#!/bin/bash # Описание: Поиск упоминаний директивы LPP_Sponsors в файлах конфигурации сборки и других связанных файлах # Ожидается: Нет результатов, если директива не используется в конфигурации сборки rg 'LPP_Sponsors' --glob '*.csproj' --glob 'Directory.Build.*' --glob '*.props' --glob '*.targets'Length of output: 98
Content.Server.Database/Migrations/Postgres/20241014210800_Delete.Designer.cs (1)
1223-1263:⚠️ Potential issueПроверьте корректность условной компиляции
Код внутри директивы
#if LPP_Sponsorsможет не компилироваться, если символLPP_Sponsorsне определён. Убедитесь, что условная компиляция настроена правильно и что удаление функционала не приведёт к ошибкам компиляции.
Content.Server.Database/Migrations/Sqlite/20241014210750_Delete.Designer.cs
Show resolved
Hide resolved
Content.Server.Database/Migrations/Postgres/20241014210800_Delete.Designer.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2 hunks)
🧰 Additional context used
🔇 Additional comments (1)
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (1)
391-392: Добавьте сообщение об ошибке в переменную 'reason' при несоответствии условиямВ методе
IsValid(строки 391-392) переменнаяreasonвсегда устанавливается вnull. Для улучшения пользовательского опыта рекомендуется предоставлять информативное сообщение, объясняющее причину отказа при несоответствии условий спонсорства.
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
|
В этом ПР'е есть конфликты, пожалуйста, устраните их, прежде чем мы сможем оценить ваш ПР. |
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2 hunks)
🧰 Additional context used
🔇 Additional comments (2)
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2)
375-394: КлассCharacterSponsorRequirementреализован корректноРеализация класса
CharacterSponsorRequirementвыглядит правильной. МетодIsValidкорректно проверяет, соответствует ли уровень спонсорства заданным параметрамMinиMax.
396-414: КлассCharacterUUIDRequirementреализован корректноРеализация класса
CharacterUUIDRequirementкорректна. МетодIsValidправильно сравнивает переданныйuuidс ожидаемымUuid.
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Logic.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirementsSystem.cs
Outdated
Show resolved
Hide resolved
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs
Outdated
Show resolved
Hide resolved
github-actions
bot
added
Изменения: Спрайт
|
RSI Diff Bot; head commit cadd2a1 merging into da1df6c Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi
Resources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi
Resources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi
Resources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi
Resources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi
Resources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi
Resources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi
Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi
Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi
Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi
Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi
Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi
|
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 (58)
Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/equipped-BREAST.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/equipped-UNDERWEAR.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/equipped-SOCKS.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/equipped-INNERCLOTHING.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/equipped-INNERCLOTHING.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/inhand-right.pngis excluded by!**/*.png
📒 Files selected for processing (42)
- .github/workflows/test-secrets4.yml (2 hunks)
- .github/workflows/validate-rsis.yml (2 hunks)
- Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/backpack.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/duffel.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/satchel.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/head/hairbows.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/breast/breast.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/socks/socks.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/underwear/underwear.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/cloaks.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/scarf.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/jumpskirts.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/corvax/sponsor.ftl (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/backpack.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/duffel.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/satchel.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Breast/breast.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Socks/socks.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Underwear/underwear.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/cloaks.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/scarf.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/jumpskirts.yml (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/meta.json (0 hunks)
💤 Files with no reviewable changes (38)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/backpack.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/duffel.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/satchel.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/head/hairbows.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/breast/breast.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/socks/socks.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/underwear/underwear.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/cloaks.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/scarf.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/jumpskirts.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/corvax/sponsor.ftl
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/backpack.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/duffel.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/satchel.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Breast/breast.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Socks/socks.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Underwear/underwear.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/cloaks.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/scarf.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/jumpskirts.yml
- Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/meta.json
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🔇 Additional comments (6)
Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1)
1-1: 🧹 Nitpick (assertive)Подтвердите намеренное удаление всего содержимого файла.
Файл теперь пуст. Убедитесь, что удаление всех сущностей (
LPPClothingHeadHatBlackHairbowsиLPPClothingHeadHatRedHairbows) было намеренным действием.Если это не было запланировано, я могу помочь восстановить удаленное содержимое или предложить альтернативную структуру файла. Нужна ли вам помощь в этом вопросе?
.github/workflows/validate-rsis.yml (2)
25-25: Улучшение безопасности установки зависимостей PythonУдаление опции
--break-system-packagesиз командыpip3 install- это правильное решение. Эта опция может потенциально вызвать конфликты с системными пакетами. Её удаление повышает безопасность и стабильность процесса установки.Выполните следующий скрипт для проверки отсутствия
--break-system-packagesв других файлах рабочих процессов:#!/bin/bash # Описание: Проверка отсутствия --break-system-packages в других файлах рабочих процессов # Тест: Поиск использования --break-system-packages в других файлах рабочих процессов rg --type yaml 'pip.*--break-system-packages' .github/workflows
16-16: Обновление версии Ubuntu для запуска задачиИзменение
runs-onсubuntu-latestнаubuntu-22.04обеспечивает более точный контроль над средой выполнения. Это может улучшить согласованность сборок, но потребует периодического обновления для поддержания актуальности.Выполните следующий скрипт для проверки использования
ubuntu-22.04в других рабочих процессах:✅ Verification successful
Проверка использования
ubuntu-22.04в других рабочих процессахПредыдущий скрипт не дал ожидаемого результата. Пожалуйста, выполните следующий скрипт для проверки использования
ubuntu-22.04в других рабочих процессах:
Использование
ubuntu-22.04подтверждено в других рабочих процессахИзменение
runs-onнаubuntu-22.04подтверждено и используется в следующих файлах рабочих процессов:
.github/workflows/test-secrets4.yml.github/workflows/validate-rsis.yml🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Проверка использования ubuntu-22.04 в других файлах рабочих процессов # Тест: Поиск использования ubuntu-22.04 в других файлах рабочих процессов rg --type yaml 'runs-on:.*ubuntu-22\.04' .github/workflowsLength of output: 65
Script:
#!/bin/bash # Описание: Проверка использования ubuntu-22.04 в других файлах рабочих процессов # Тест: Поиск использования runs-on: ubuntu-22.04 в других файлах рабочих процессов rg --type yaml 'runs-on:\s*ubuntu-22\.04' .github/workflowsLength of output: 185
.github/workflows/test-secrets4.yml (2)
30-30: Улучшение безопасности установки пакетов PythonУдаление флага
--break-system-packagesиз командыpip install- это правильное решение. Этот флаг может вызвать конфликты с системными пакетами.Выполните следующий скрипт, чтобы убедиться, что установка пакетов все еще работает без этого флага:
#!/bin/bash # Описание: Проверка успешной установки пакетов без флага --break-system-packages # Проверка: Попытка установки пакетов без флага --break-system-packages pip3 install --ignore-installed --user pillow jsonschema # Проверка: Вывод установленных версий пакетов pip3 show pillow jsonschema
13-13: Обновление версии Ubuntu для выполнения задачиИзменение
runs-onсubuntu-latestнаubuntu-22.04обеспечивает более точную спецификацию среды выполнения. Это может улучшить воспроизводимость сборки, но также означает, что вы не будете автоматически получать обновления Ubuntu.Выполните следующий скрипт, чтобы проверить использование
ubuntu-22.04в других рабочих процессах:✅ Verification successful
Проверка использования ubuntu-22.04 во всех рабочих процессах
Изменение
runs-onнаubuntu-22.04подтверждено в следующих файлах рабочих процессов:
.github/workflows/validate-rsis.yml.github/workflows/test-secrets4.ymlЭто обеспечивает консистентность среды выполнения и не приводит к конфликтам.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Проверка использования ubuntu-22.04 в других файлах рабочих процессов # Проверка: Поиск использования ubuntu-22.04 в других файлах рабочих процессов rg --type yaml 'runs-on:.*ubuntu-22\.04' .github/workflowsLength of output: 232
Script:
#!/bin/bash # Описание: Проверка использования ubuntu-22.04 в других файлах рабочих процессов # Проверка: Поиск использования runs-on с ubuntu-22.04 в файлах рабочих процессов rg --type yaml 'runs-on:\s*ubuntu-22\.04' .github/workflowsLength of output: 185
Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (1)
366-372: Проверьте корректность передачи параметраsponsorTierУбедитесь, что параметр
sponsorTierпередается в методIsValidс фактическим уровнем спонсорства пользователя. Если всегда передается значение по умолчанию0, проверка может работать некорректно.Чтобы убедиться в правильности передачи параметра, выполните следующий скрипт:
There was a problem hiding this comment.
Actionable comments posted: 5
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
⛔ Files ignored due to path filters (58)
Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/equipped-BACKPACK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/equipped-HELMET.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/equipped-NECK.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/equipped-BREAST.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/equipped-UNDERWEAR.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/equipped-SOCKS.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/equipped-INNERCLOTHING.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/equipped-INNERCLOTHING.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/inhand-right.pngis excluded by!**/*.png
📒 Files selected for processing (42)
- .github/workflows/test-secrets4.yml (2 hunks)
- .github/workflows/validate-rsis.yml (2 hunks)
- Content.Shared/Customization/Systems/CharacterRequirements.Profile.cs (2 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/backpack.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/duffel.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/satchel.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/head/hairbows.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/breast/breast.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/socks/socks.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/underwear/underwear.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/cloaks.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/scarf.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/jumpskirts.ftl (0 hunks)
- Resources/Locale/ru-RU/ss14-ru/prototypes/corvax/sponsor.ftl (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/backpack.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/duffel.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/satchel.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Breast/breast.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Socks/socks.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Underwear/underwear.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/cloaks.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/scarf.yml (0 hunks)
- Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/jumpskirts.yml (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/meta.json (0 hunks)
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/meta.json (0 hunks)
💤 Files with no reviewable changes (38)
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/backpack.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/duffel.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/back/satchel.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/head/hairbows.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/breast/breast.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/socks/socks.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/lpp/underwear/underwear.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/cloaks.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/neck/scarf.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/clothing/uniforms/jumpskirts.ftl
- Resources/Locale/ru-RU/ss14-ru/prototypes/corvax/sponsor.ftl
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/backpack.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/duffel.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Back/satchel.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Breast/breast.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Socks/socks.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/LPP/Underwear/underwear.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/cloaks.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Neck/scarf.yml
- Resources/Prototypes/_LostParadise/Entities/Clothing/Uniforms/jumpskirts.yml
- Resources/Textures/_LostParadise/Clothing/Back/Backpacks/blackbackpack.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Back/Duffels/blackduffel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Back/Satchels/blacksatchel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/blackhairbows.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Head/HairBows/redhairbows.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Neck/Cloaks/Monorich.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Neck/Scarfs/windy-scarf.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/Cargocloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/engineeringcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/medicalcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/scientificcloakhel.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/OuterClothing/Cloak/securitycloak.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowbra.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowpanties.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Underwear/cowset/cowsocks.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/altwhitetopic.rsi/meta.json
- Resources/Textures/_LostParadise/Clothing/Uniforms/Topics/whitetopic.rsi/meta.json
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1)
Pattern
**/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу
prob:, можно оставить оригинальную часть с- id:.
🔇 Additional comments (3)
Resources/Prototypes/_LostParadise/Entities/Clothing/Head/hairbows.yml (1)
1-1: Подтвердите намеренное удаление сущностей и проверьте влияние на системуФайл теперь пуст, что указывает на полное удаление сущностей
LPPClothingHeadHatBlackHairbowsиLPPClothingHeadHatRedHairbows. Пожалуйста, подтвердите, что это намеренное изменение.Рекомендую выполнить следующие действия:
- Убедитесь, что удаление этих сущностей не оказывает негативного влияния на другие части системы.
- Проверьте, что эти предметы правильно обрабатываются в новой вкладке "спонсорские предметы", упомянутой в целях PR.
✅ Verification successful
Удаление сущностей подтверждено и не влияет на систему
В ходе проверки не были найдены ссылки на
LPPClothingHeadHatBlackHairbowsиLPPClothingHeadHatRedHairbows, что подтверждает их намеренное удаление и отсутствие влияния на другие части системы.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Описание: Поиск ссылок на удаленные сущности в кодовой базе echo "Поиск ссылок на LPPClothingHeadHatBlackHairbows:" rg "LPPClothingHeadHatBlackHairbows" --type yaml echo "Поиск ссылок на LPPClothingHeadHatRedHairbows:" rg "LPPClothingHeadHatRedHairbows" --type yaml echo "Поиск возможных новых местоположений этих сущностей:" fd -e yml -e yaml | xargs grep -i "hairbows"Length of output: 465
.github/workflows/validate-rsis.yml (1)
25-25: Одобрено удаление опции --break-system-packages.Удаление опции
--break-system-packagesиз командыpip3 install- это правильное решение. Эта опция может быть опасной, так как позволяет pip изменять системные пакеты.Текущая реализация более безопасна и соответствует лучшим практикам использования pip в CI/CD окружении.
.github/workflows/test-secrets4.yml (1)
30-30: Улучшена команда установки зависимостей Python.Изменения в команде
pip installповышают безопасность и изоляцию установки пакетов:
- Удаление опции
--break-system-packagesпредотвращает потенциальные конфликты с системными пакетами.- Добавление
--ignore-installedи--userобеспечивает установку пакетов в домашний каталог пользователя, что безопаснее и более изолированно.Эти изменения улучшают управление зависимостями и снижают риск конфликтов с системными пакетами.
🆑 Evgencheg