[Remove] No more pride #1027
[Remove] No more pride #1027
Conversation
|
""" WalkthroughВ данном изменении скорректированы определения сущностей, связанных с прайдом. В файле Changes
Sequence Diagram(s)sequenceDiagram
participant S as RandomVendingSpawner
participant T as ClothingVendorTable
participant M as VendingMachine
S->>T: Запрос списка вариантов
T-->>S: Отправка списка (без VendingMachinePride)
S->>S: Выбор доступного варианта
S->>M: Создание торгового автомата по выбору
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 300000ms (3)
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
S: Untriaged
size/S
No C#
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
Resources/Prototypes/Entities/Markers/Spawners/Random/vending.yml(1 hunks)Resources/Prototypes/Loadouts/loadout_groups.yml(1 hunks)Resources/Prototypes/holidays.yml(3 hunks)Resources/migration.yml(1 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/migration.yml
[error] 746-746: duplication of key "ClothingNeckNonBinaryPin" in mapping
(key-duplicates)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: Test Packaging
- GitHub Check: YAML Linter
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (4)
Resources/Prototypes/Entities/Markers/Spawners/Random/vending.yml (1)
48-49: Удаление элемента VendingMachinePride
Изменения в данном блоке комментируют строку с идентификаторомVendingMachinePrideв таблицеClothingVendorTable. Это соответствует цели PR по исключению контента, связанного с прайдом, и изменяет логику выбора торговых автоматов. Рекомендуется проверить, что отсутствует зависимость в других частях системы, где мог использоваться этот элемент.Resources/Prototypes/holidays.yml (1)
1-447: Массовое удаление праздничных записей
В этом файле удалены определения множества праздников – как общих, так и посвящённых тематике прайда (например,NewYear,MisterLizard,ChineseNewYear,ValentinesDayи другие), а также некоторые праздники, связанные с ЛГБТК+ тематикой, закомментированы. Изменения соответствуют требованиям PR по исключению запрещённого контента. Рекомендуется удостовериться, что отсутствие этих записей не приведёт к ошибкам при выборе или обработке праздников в игровом процессе.Resources/migration.yml (1)
735-760: Удаление идентификаторов прайд-тематики
В этом фрагменте файла задаются значенияnullдля множества идентификаторов, связанных с ЛГБТК+ тематикой (например,LGBTQFlag,ClothingNeckLGBTPin,ClothingNeckAromanticPinи т.д.), а также дляVendingMachinePride. Эти изменения корректно реализуют требование PR по исключению соответствующего контента. Проверьте, что такое удаление не нарушает обратную совместимость и не вызывает ошибок при загрузке маппинга сущностей.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 746-746: duplication of key "ClothingNeckNonBinaryPin" in mapping
(key-duplicates)
Resources/Prototypes/Loadouts/loadout_groups.yml (1)
22-33: Удаление элементов прайд-тематики из группы "Trinkets"
В секции группыTrinketsблок, содержащий элементы вродеClothingNeckLGBTPin,ClothingNeckAromanticPinи т.д., был закомментирован. Это соответствует общему направлению PR по удалению прайд-контента. Рекомендуется удостовериться, что данное изменение не повлияет на логику формирования загрузочных групп и корректную работу системы.
Resources/migration.yml
Outdated
| ClothingNeckNonBinaryPin: null | ||
| ClothingNeckNonBinaryPin: null |
There was a problem hiding this comment.
Критическая ошибка: Дублирование ключа "ClothingNeckNonBinaryPin"
Обнаружено повторение ключа ClothingNeckNonBinaryPin на строках 745 и 746. Это может привести к конфликтам при парсинге YAML. Рекомендуется удалить один из дублирующихся ключей.
🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 746-746: duplication of key "ClothingNeckNonBinaryPin" in mapping
(key-duplicates)
There was a problem hiding this comment.
Actionable comments posted: 0
🔭 Outside diff range comments (1)
Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml (1)
100-107: Критическая ошибка: ссылки на удалённые сущности Cloak
В группе «Cloaks» таблицы настроек присутствуют ссылки на сущности, такие как ClothingNeckCloakAce, ClothingNeckCloakAro, ClothingNeckCloakBi, ClothingNeckCloakIntersex, ClothingNeckCloakLesbian, ClothingNeckCloakGay, ClothingNeckCloakEnby и ClothingNeckCloakPan. Так как данные сущности были удалены или закомментированы в файле cloaks.yml, это приводит к ошибке (см. ошибку YAML Linter). Необходимо удалить или обновить эти ссылки.Предлагаемое исправление:
- - id: ClothingNeckCloakAce - - id: ClothingNeckCloakAro - - id: ClothingNeckCloakBi - - id: ClothingNeckCloakIntersex - - id: ClothingNeckCloakLesbian - - id: ClothingNeckCloakGay - - id: ClothingNeckCloakEnby - - id: ClothingNeckCloakPan🧰 Tools
🪛 GitHub Actions: YAML Linter
[error] 105-105: No EntityPrototype found with id ClothingNeckCloakAce
🧹 Nitpick comments (3)
Resources/Prototypes/Entities/Objects/Misc/handy_flags.yml (1)
37-47: Удаление записи LGBTQHandyFlag через комментарий.
В этом блоке код для сущности LGBTQHandyFlag закомментирован (строки 37-47), что соответствует цели пул-реквеста по исключению контента с признавательным подтекстом. Рекомендую рассмотреть один из вариантов:
- Если элемент больше не планируется использовать, полностью удалите данный блок, чтобы избежать путаницы в будущем.
- Если оставление кода в виде комментария оправдано (например, для исторической справки), добавьте более подробное пояснение относительно причин такого решения.
Также рекомендуется использовать единообразный язык комментариев – в остальном файле комментарии могут быть на русском, поэтому целесообразно привести этот блок к тому же стилю.Resources/Prototypes/Entities/Structures/Wallmounts/Signs/flags.yml (1)
39-47: Удаление сущности LGBTQFlag
Изменения демонстрируют, что сущность LGBTQFlag теперь закомментирована, что соответствует цели PR по удалению элементов, связанных с прайдом. Если это окончательное решение, рекомендуется рассмотреть вариант полного удаления этого блока, чтобы улучшить читаемость и поддерживаемость файла, избавившись от неиспользуемого кода.Resources/Prototypes/Entities/Structures/Machines/vending_machines.yml (1)
2156-2183: Добавление автоматов для выдачи газовых баллонов
Добавлены две новые сущности – VendingMachineTankDispenserEVA и VendingMachineTankDispenserEngineering, предназначенные для выдачи газовых баллонов. Обратите внимание, что в комментариях указано доработать визуализатор оставшихся баллонов ("#TODO add visualiser for remaining tanks as layers").
Рекомендуется проверить корректность ссылок на инвентарь (TankDispenserEVAInventory и TankDispenserEngineeringInventory) и убедиться, что новый функционал интегрирован с интерфейсом пользователя.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (13)
Resources/Prototypes/Catalog/VendingMachines/Inventories/pride.yml(1 hunks)Resources/Prototypes/Entities/Clothing/Neck/cloaks.yml(1 hunks)Resources/Prototypes/Entities/Clothing/Neck/pins.yml(1 hunks)Resources/Prototypes/Entities/Clothing/Neck/scarfs.yml(1 hunks)Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml(1 hunks)Resources/Prototypes/Entities/Objects/Misc/handy_flags.yml(1 hunks)Resources/Prototypes/Entities/Structures/Furniture/dresser.yml(1 hunks)Resources/Prototypes/Entities/Structures/Machines/nuke.yml(2 hunks)Resources/Prototypes/Entities/Structures/Machines/vending_machines.yml(1 hunks)Resources/Prototypes/Entities/Structures/Wallmounts/Signs/flags.yml(1 hunks)Resources/Prototypes/Loadouts/Miscellaneous/trinkets.yml(1 hunks)Resources/Prototypes/Loadouts/loadout_groups.yml(1 hunks)Resources/migration.yml(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- Resources/Prototypes/Catalog/VendingMachines/Inventories/pride.yml
- Resources/Prototypes/Entities/Clothing/Neck/pins.yml
🧰 Additional context used
🪛 GitHub Actions: YAML Linter
Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml
[error] 105-105: No EntityPrototype found with id ClothingNeckCloakAce
⏰ 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 (13)
Resources/Prototypes/Entities/Clothing/Neck/scarfs.yml (1)
133-242: Комментарий: Удаление или временное отключение pride-шарфовВ данном блоке закомментированы все определения pride-шарфов, что соответствует заявленной цели PR по удалению незаконного и запрещённого контента. Однако закомментированный код остаётся в репозитории и может снижать читаемость и чистоту файла. Рекомендуется, если данные элементы более не требуются, полностью удалить их из файла, что позволит избежать путаницы при дальнейшем сопровождении проекта. Также убедитесь, что никакие внешние ссылки или зависимости в системе не ссылаются на эти определения.
Resources/Prototypes/Entities/Structures/Machines/nuke.yml (3)
30-31: Проверка отключения праздничных визуалов для pride
В этом участке закомментированы строки, отвечающие за отображение визуалов pride (в HolidayVisuals). Изменение соответствует заявленной цели удаления данного контента. Убедитесь, что ни в одном другом компоненте не ссылаются на устаревшие идентификаторы.
37-38: Проверка конфигурации GenericVisualizer
Обратите внимание, что в блоке GenericVisualizer остаётся настройка для pride (visible: true). Это может создавать рассогласование с закомментированными настройками в HolidayVisuals. Рекомендуется перепроверить, соответствует ли данное поведение общей концепции удаления контента pride.
150-151: Проверка изменения визуалов для NuclearBombKeg
Аналогично предыдущему, здесь строки для pride в разделе HolidayVisuals закомментированы. Убедитесь, что все ссылки на эти визуалы во всём проекте корректно обновлены и не вызывают конфликтов.Resources/Prototypes/Entities/Clothing/Neck/cloaks.yml (1)
211-282: Удаление сущностей LGBTQ+ (Cloak)
В этом блоке закомментированы определения нескольких сущностей (ClothingNeckCloakAce, ClothingNeckCloakAro, ClothingNeckCloakBi, ClothingNeckCloakIntersex, ClothingNeckCloakLesbian, ClothingNeckCloakGay, ClothingNeckCloakEnby и ClothingNeckCloakPan). Это соответствует цели PR по исключению соответствующего контента. Проверьте, что данные сущности не используются в других местах конфигураций.Resources/Prototypes/Loadouts/Miscellaneous/trinkets.yml (2)
103-180: Удаление loadout-записей для LGBTQ+ пинов
В данном участке закомментированы записи для пинов, таких как ClothingNeckLGBTPin, ClothingNeckAromanticPin, ClothingNeckAsexualPin, ClothingNeckBisexualPin, ClothingNeckGayPin, ClothingNeckIntersexPin, ClothingNeckLesbianPin, ClothingNeckNonBinaryPin, ClothingNeckPansexualPin, ClothingNeckOmnisexualPin, ClothingNeckTransPin, ClothingNeckAutismPin и ClothingNeckGoldAutismPin. Это изменение соответствует цели удаления ненужного контента.
181-188: Проверка записей для Backmen loadouts
Новые записи для Backmen (например, ClothingNeckDonkPin, ClothingNeckDeForestPin и прочие) добавлены корректно и не зависят от удалённых LGBTQ+ пинов. Рекомендуется убедиться, что их конфигурация не приводит к логическим конфликтам в общем списке лута.Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml (1)
143-155: Удаление ссылок на LGBTQ+ пины в MaintFluffTable
В этом блоке ссылки на пины (например, ClothingNeckLGBTPin и пр.) закомментированы, что соответствует общей стратегии удаления этих элементов. Рекомендуется проверить, что связанные зависимости в других таблицах и спаунерах также обновлены.Resources/Prototypes/Entities/Structures/Furniture/dresser.yml (2)
58-60: Отключение элемента ClothingNeckLGBTPin в DresserFilled
В компоненте StorageFill сущности DresserFilled закомментирован элемент ClothingNeckLGBTPin. Это изменение соответствует цели PR по удалению нежелательного контента. Проверьте, что изменение не нарушает баланс вероятностей распределения лута.
61-555: Проверка конфигурации наполнения DresserFilled
Большой объем конфигурации заполнения (loot) DresserFilled требует внимательной проверки. Убедитесь, что удаление элемента ClothingNeckLGBTPin и аналогичных записей корректно отражено в расчётах вероятностей и не приводит к рассогласованию распределения оставшегося лута.Resources/migration.yml (1)
735-759: Удаление элементов прайд-тематики в миграции
Удалены записи, связанные с элементами, отражающими прайд (например, LGBTQFlag, LGBTQHandyFlag, ClothingNeckLGBTPin, ClothingNeckAromanticPin, ClothingNeckAsexualPin, ClothingNeckBisexualPin, ClothingNeckGayPin, ClothingNeckIntersexPin, ClothingNeckLesbianPin, ClothingNeckNonBinaryPin, ClothingNeckPansexualPin, ClothingNeckOmnisexualPin, ClothingNeckTransPin, а также связанные с шарфами и VendingMachinePride). Эти изменения соответствуют цели PR "[Tweak] No more pride".
Пожалуйста, убедитесь, что данное удаление согласовано с архитектурой системы и что любые ссылки на удалённые элементы корректно обработаны в других частях проекта.Resources/Prototypes/Loadouts/loadout_groups.yml (1)
22-35: Удаление прайд-значков из группы "Trinkets"
Закомментированы элементы, связанные с прайдом (например, ClothingNeckLGBTPin, ClothingNeckAromanticPin, ClothingNeckAsexualPin и т.д.) в группе загрузок "Trinkets". Данный подход соответствует поставленной задаче по исключению прайд-тематики. Проверьте, что изменения не влияют на остальные элементы группы и что все зависимости корректно обработаны.Resources/Prototypes/Entities/Structures/Machines/vending_machines.yml (1)
2123-2153: Удаление определения Pride-O-Mat
В этом блоке закомментировано определение сущности Pride-O-Mat (ранее отвечающей за прайд-автомат). Такое удаление согласуется с общей стратегией PR по исключению прайд-темой из системы. Убедитесь, что все зависимости и ссылки на данный автомат обновлены или удалены, чтобы избежать ошибок при загрузке карты или вызовах интерфейса.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/migration.yml (1)
734-734: Комментарий к отметке изменений
На строке 734 виден комментарий «#2025-02-06 backmen: no pride», который обозначает причину удаления элементов, связанных с прайдом, в соответствии с целями PR. Рекомендуется, по возможности, добавить подробное пояснение в документации миграции для будущего понимания этих изменений.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/migration.yml(1 hunks)
⏰ 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/migration.yml (3)
735-744: Удаление основных ключей Pride (часть 1)
В строках 735–744 устанавливаются значения null для следующих ключей:
• LGBTQFlag
• LGBTQHandyFlag
• ClothingNeckLGBTPin
• ClothingNeckAromanticPin
• ClothingNeckAsexualPin
• ClothingNeckBisexualPin
• ClothingNeckGayPin
• ClothingNeckIntersexPin
• ClothingNeckLesbianPin
• ClothingNeckNonBinaryPin
Это соответствует удалению соответствующих сущностей из миграционного маппинга. Убедитесь, что все ссылки на эти ключи в системе обновлены и что данное удаление согласуется с общей бизнес-логикой.
745-757: Удаление дополнительных ключей Pride (часть 2)
В строках 745–757 продолжается установка значения null для остальных ключей, связанных с темой прайда, а именно:
• ClothingNeckPansexualPin
• ClothingNeckOmnisexualPin
• ClothingNeckTransPin
• ClothingNeckAutismPin
• ClothingNeckGoldAutismPin
• ClothingNeckScarfStripedAce
• ClothingNeckScarfStripedAro
• ClothingNeckScarfStripedBiSexual
• ClothingNeckScarfStripedGay
• ClothingNeckScarfStripedInter
• ClothingNeckScarfStripedLesbian
• ClothingNeckScarfStripedPan
• ClothingNeckScarfStripedNonBinary
• ClothingNeckScarfStripedRainbow
• ClothingNeckScarfStripedTrans
Важно проверить, что после удаления этих ключей YAML-файл остаётся корректным и не возникает конфликтов или дублирования идентификаторов.
758-768: Удаление оставшихся ключей и VendingMachinePride
В строках 758–768 значения null назначаются для следующих элементов:
• ClothingNeckCloakAce
• ClothingNeckCloakAro
• ClothingNeckCloakBi
• ClothingNeckCloakIntersex
• ClothingNeckCloakLesbian
• ClothingNeckCloakGay
• ClothingNeckCloakEnby
• ClothingNeckCloakPan
• VendingMachinePride
Это окончательно исключает из маппинга соответствующие записи, что соответствует заявленным целям PR по удалению запрещённого контента. Рекомендуется убедиться, что зависимые системы корректно обработают отсутствие этих сущностей.
github-actions
bot
added
the
Changes: Map
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/Entities/Clothing/Neck/cloaks.yml (1)
138-282: Закомментированные сущности, связанные с тематикой прайда.
В этом диапазоне закомментированы определения элементов, соответствующих прайд-тематике (например, плащ для вампиров, плащ пилота, плащ волка и т.д.). Если элемент не планируется к дальнейшему использованию, рекомендуется полностью удалить эти блоки вместо их комментирования. Это поможет упростить поддержку и чтение файла.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
Resources/Prototypes/Entities/Clothing/Neck/cloaks.yml(2 hunks)Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml(2 hunks)Resources/Prototypes/Entities/Objects/Decoration/present.yml(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- Resources/Prototypes/Entities/Markers/Spawners/Random/maintenance.yml
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (2)
Resources/Prototypes/Entities/Clothing/Neck/cloaks.yml (1)
1-137: Удаление основных сущностей плащей.
В этом блоке были удалены определения множества сущностей (например, плащи для центрального командира, капитана, главы безопасности и т.д.). Проверьте, что ссылки на эти сущности в других файлах (например, в миграционных маппингах или таблицах спавна) также обновлены, чтобы не возникло ошибок при загрузке данных.Resources/Prototypes/Entities/Objects/Decoration/present.yml (1)
197-199: Удаление элемента из списка спавна подарков.
Здесь элемент с id "ClothingNeckCloakTrans" закомментирован в блоке items компонента SpawnItemsOnUse. Это изменение соответствует общей политике удаления запрещённого или нежелательного контента. Убедитесь, что данное изменение согласовано с остальными частями системы (например, миграционными данными и настройками вендоматов).
вырезано всё незаконное и запрещённейшее
Summary by CodeRabbit