повесить фото на 173#993
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughSCP-173 получил систему инвентаря с поддержкой масок (шаблон scp173). Разработана носимая версия фото SCP-096 как предмет маски с компонентами Clothing/ScpMask; соответствующие RSI-файлы метаданных и шаблон инвентаря добавлены или реорганизованы. ChangesИнвентарная система SCP-173 и переоборудование SCP-096 фото
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 10✅ Passed checks (10 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![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
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@Resources/Prototypes/_Scp/InventoryTemplates/scp173.yml`:
- Around line 1-9: Prototype identifier is not CamelCase: change the id value in
the inventory template from "scp173" to "Scp173" and update any corresponding
templateId references (notably in the entity file that currently uses
templateId: scp173 in
Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp173.yml) so they
match the new CamelCase identifier; ensure all references across the project
that point to id: scp173 are updated to Scp173 to keep names consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 74f3daab-1c08-45e7-b885-e7efc4473a63
⛔ Files ignored due to path filters (3)
Resources/Textures/_Scp/Objects/Research/Scp096/Clothing.rsi/equipped-MASK.pngis excluded by!**/*.pngResources/Textures/_Scp/Objects/Research/Scp096/Photo.rsi/scp096-photo-undeveloped.pngis excluded by!**/*.pngResources/Textures/_Scp/Objects/Research/Scp096/Photo.rsi/scp096-photo.pngis excluded by!**/*.png
📒 Files selected for processing (7)
Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp173.ymlResources/Prototypes/_Scp/Entities/Objects/ScpParts/parts.ymlResources/Prototypes/_Scp/InventoryTemplates/scp173.ymlResources/Textures/_Scp/Mobs/Scp/scp-173.rsi/.scp-173.png-autosave.kraResources/Textures/_Scp/Objects/Research/Scp096/Clothing.rsi/meta.jsonResources/Textures/_Scp/Objects/Research/Scp096/Photo.rsi/meta.jsonResources/Textures/_Scp/Objects/Research/research_material.rsi/meta.json
💤 Files with no reviewable changes (1)
- Resources/Textures/_Scp/Objects/Research/research_material.rsi/meta.json
|
Абсолютно НИКТО не осматривает статую, а чтобы "трюк" сработал - нужно очень много везения. Может сделать механ как с прямым взглядом на 096? |
разве это не будет тогда слишком сильно? |
в любом случае я не умею в C#, поэтому это можно просто оставить как начальную базу чтобы потом уже доделать что нужно |
|
Будто бы будет очень дизбалансно, дешка вешает фотку на 173 и всё... Кодом то конечно это реализуемо, но контрить это будет тяжко, разве что с помощью обскуров... |
|
Это стоит под очень большим вопросом. |
096 триггерится только если осмотришь 173 с фоткой |
|
это не супер пупер имба |
Если делать через осмотр - то получиться не реалистичная фигня, а если делать через код, что когда просто посмотрел - то это слишком имба, хоть это и можно контрить обскурами. |
|
@666XxttimurkaxX666 подумал там? |
чо |
поставь пока что просто в драфт я хз либо замержи ибо оно как бы ничего сильно и не меняет но и не помешало бы в будущем |
Так вроде бы обговорили, что это могут делать только антаги. Можно просто сделать для них отдельный предмет скотча. Используют его на фотке и теперь фотку можно крепить на 173. Как пример. |
|
Этот PR содержит конфликты, пожалуйста, разрешите их, прежде чем мы сможем его оценить. |
а сосисоси за 5 баксов не сделаешь |
4 participants
Краткое описание | Short description
ну типо можно повесить фото 096 на 173
если что на скриншоте фотка хуевая из за того что я тогда на время включил низкое разрешение
Ссылка на багрепорт/Предложение | Related Issue/Bug Report
Медиа (Видео/Скриншоты) | Media (Video/Screenshots)
Changelog
🆑 mama drd
Summary by CodeRabbit