[TWEAK] Thuster and dynamic chaos, drask#2614
[TWEAK] Thuster and dynamic chaos, drask#2614CrimeMoot wants to merge 5 commits intoAdventureTimeSS14:masterfrom
Conversation
WalkthroughДобавлена новая маркировка Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip CodeRabbit can use OpenGrep to find security vulnerabilities and bugs across 17+ programming languages.OpenGrep is compatible with Semgrep configurations. Add an |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 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/ADT/dynamic_weights.yml`:
- Around line 4-5: Вы удалили DynamicLight только из таблицы весов; нужно
полностью удалить/деактивировать сам пресет "DynamicLight" и связанное правило
"DynamicLightGamerule" — найдите объявление пресета DynamicLight и блок правила
DynamicLightGamerule и либо удалите их, либо пометьте как неактивные, и затем
убедитесь, что больше нигде не ссылаются на эти символы (например в списках
режимов или вызовах), чтобы режим не мог быть вызван вне weightedRandom.
In `@Resources/Prototypes/ADT/Entities/Mobs/Customization/Markings/Drask.yml`:
- Around line 83-92: В записи маркировки FaceDraskBase исправьте путь к RSI в
секции sprites: замените обратные слеши в значении sprite
(ADT\Mobs\Species\Drask\parts.rsi) на прямые слеши, чтобы он совпадал со стилем
остальных путей (ADT/Mobs/Species/Drask/parts.rsi); проверьте поле sprites и
значение sprite в сущности FaceDraskBase, чтобы загрузчик корректно нашёл
дефолтную метку.
In `@Resources/Prototypes/ADT/Species/drask.yml`:
- Around line 89-91: Слот Snout сейчас блокируется базовой меткой FaceDraskBase
потому что в блоке с атрибутами (points: 1, required: true, defaultMarkings: [
FaceDraskBase ]) обязательная метка занимает единственный слот и не даёт выбрать
FaceDrask / FaceDraskBig / FaceDraskDouble; исправьте это либо убрав required:
true, либо увеличив points с 1 до 2 (чтобы базовая метка + дополнительная морда
умещались), либо переместив FaceDraskBase из defaultMarkings в конфигурацию, не
резервирующую слот (если есть поддержка отдельного поля для нефайковых
дефолтов); правку внесите в тот же блок, где заданы points, required и
defaultMarkings (упомянутые символы: Snout, points, required, defaultMarkings,
FaceDraskBase, FaceDrask, FaceDraskBig, FaceDraskDouble).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: e0c3be65-426e-4ded-9c83-c3ed7f69b9db
📒 Files selected for processing (5)
Resources/Prototypes/ADT/Entities/Mobs/Customization/Markings/Drask.ymlResources/Prototypes/ADT/Entities/Objects/Devices/Circuitboards/Machine/production.ymlResources/Prototypes/ADT/GameRules/roundstart.ymlResources/Prototypes/ADT/Species/drask.ymlResources/Prototypes/ADT/dynamic_weights.yml
|
В чейнджлоге поменяй "ХОАС" на "ХАОС" |
|
Про cl забыл |
Точно |
| maxChaos: 60 | ||
| minChaos: 50 | ||
| maxChaos: 80 | ||
| minChaos: 70 |
There was a problem hiding this comment.
По решению руководства нужно откатить до 50 очков
There was a problem hiding this comment.
До 50 очков чего. Мин или Макс?
Описание PR
Снизил стоимость двигателей, поменял крафт - требует обычные БС кристаллы, а не исскуственные. Убрал лайт динамик, повысил медиум динамик до 80 по просьбе. Починил драсков
Техническая информация
Чейнджлог
🆑 CrimeMoot