[Feature] Added More Slime People Markings #345
Conversation
# Conflicts: # Resources/Locale/ru-RU/corvax/markings/slime_cat.ftl # Resources/Prototypes/Corvax/slime_cat_parts.yml # Resources/Prototypes/Corvax/slime_fox_parts.yml # Resources/Prototypes/Species/slime.yml
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
WalkthroughИзменения в этом pull request в основном сосредоточены на расширении возможностей кастомизации для вида SlimePerson. Были добавлены новые маркировки для ушей и хвостов, включая варианты для SlimeCat и SlimeFox. Также обновлены ограничения на виды для существующих маркировок pointy ears, чтобы включить SlimePerson. Кроме того, были внесены изменения в метаданные текстуры для шлема capspace. Changes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (1)Resources/Textures/Clothing/Head/Hardsuits/capspace.rsi/meta.json (1)
Новые состояния шлема следуют установленному шаблону именования и содержат правильное количество направлений (4), как и другие видоспецифичные состояния шлема. Давайте проверим наличие текстур для новых состояний: ✅ Verification successfulТекстуры для состояний шлема вокса присутствуют! Проверка подтвердила наличие всех необходимых файлов текстур:
🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Проверка наличия файлов текстур для новых состояний шлема
# Поиск файлов текстур для новых состояний
fd "off-equipped-HELMET-vox" "Resources/Textures/Clothing/Head/Hardsuits/capspace.rsi"
fd "on-equipped-HELMET-vox" "Resources/Textures/Clothing/Head/Hardsuits/capspace.rsi"
Length of output: 335 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
Изменения: Спрайт
Signed-off-by: Remuchi <[email protected]>
|
RSI Validator test fail unrelated to my PR |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (14)
Resources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_cat_inner.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_cat_outer.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_curled_inner.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_curled_outer.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_stubby_inner.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_stubby_outer.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_torn_inner.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/ears_slime_torn_outer.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/slime_tail_cat.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/slime_tail_cat_wag.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/slime_tail_cat_wag_stripes_prime.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_cat_parts.rsi/slime_tail_cat_wag_stripes_second.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_fox_parts.rsi/ears_slime_fox_inner.pngis excluded by!**/*.pngResources/Textures/_LostParadise/Mobs/Customization/slime_fox_parts.rsi/ears_slime_fox_outer.pngis excluded by!**/*.png
📒 Files selected for processing (4)
Resources/Prototypes/Entities/Mobs/Customization/Markings/pointy_ears.yml(6 hunks)Resources/Prototypes/Species/slime.yml(2 hunks)Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_cat_parts.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_fox_parts.yml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
Resources/Prototypes/Species/slime.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_fox_parts.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Entities/Mobs/Customization/Markings/pointy_ears.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_cat_parts.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
🪛 yamllint (1.35.1)
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_fox_parts.yml
[warning] 6-6: too few spaces before comment
(comments)
[error] 8-8: wrong indentation: expected at least 3
(indentation)
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_cat_parts.yml
[warning] 6-6: too few spaces before comment
(comments)
[error] 8-8: wrong indentation: expected at least 3
(indentation)
[warning] 18-18: too few spaces before comment
(comments)
[error] 20-20: wrong indentation: expected at least 3
(indentation)
[warning] 21-21: too few spaces before comment
(comments)
[warning] 28-28: too few spaces before comment
(comments)
[error] 30-30: wrong indentation: expected at least 3
(indentation)
[warning] 40-40: too few spaces before comment
(comments)
[error] 42-42: wrong indentation: expected at least 3
(indentation)
[warning] 52-52: too few spaces before comment
(comments)
[error] 54-54: wrong indentation: expected at least 3
(indentation)
[warning] 64-64: too few spaces before comment
(comments)
🔇 Additional comments (4)
Resources/Prototypes/Entities/Mobs/Customization/Markings/pointy_ears.yml (1)
6-6: Корректное расширение ограничений видов для маркировок
Добавление SlimePerson в speciesRestriction для базовых маркировок ушей логично и соответствует целям PR. Маркировки, специфичные для Oni, правильно оставлены без изменений.
Предлагаю проверить совместимость существующих спрайтов с моделью SlimePerson:
Also applies to: 16-16, 26-26, 36-36, 46-46, 66-66
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_cat_parts.yml (1)
66-69: 🛠️ Refactor suggestion
Проверьте отступы в последней маркировке
В маркировке SlimeCatTailStripes используются отступы в 4 пробела для спрайтов, в то время как в остальных маркировках используются 2 пробела.
Примените следующие изменения для соблюдения единого стиля:
sprites:
- - sprite: _LostParadise/Mobs/Customization/slime_cat_parts.rsi
- state: slime_tail_cat_wag_stripes_prime
- - sprite: _LostParadise/Mobs/Customization/slime_cat_parts.rsi
- state: slime_tail_cat_wag_stripes_second
+ - sprite: _LostParadise/Mobs/Customization/slime_cat_parts.rsi
+ state: slime_tail_cat_wag_stripes_prime
+ - sprite: _LostParadise/Mobs/Customization/slime_cat_parts.rsi
+ state: slime_tail_cat_wag_stripes_secondLikely invalid or redundant comment.
Resources/Prototypes/Species/slime.yml (2)
25-27: Корректное добавление новых спрайтов!
Добавление новых спрайтов для HeadTop, HeadSide и Tail выполнено правильно:
- Использован корректный тип MobHumanoidAnyMarking
- Комментарии на английском языке соответствуют требованиям
- Изменения соответствуют целям PR по добавлению ушей и хвостов
47-55: 🧹 Nitpick (assertive)
Корректное добавление новых точек маркировки!
Добавление новых точек маркировки выполнено правильно:
- Значения points и required соответствуют существующему паттерну
- Комментарии на английском языке
- Структура соответствует остальным маркировкам
Небольшое предложение по улучшению:
Для лучшей организации кода, предлагаю сгруппировать связанные маркировки вместе. Например:
HeadTop:
points: 1 # Lost Paradise Sponsors
required: false
HeadSide:
points: 1 # Lost Paradise Elf Ears
required: false
+ Tail:
+ points: 1 # Lost Paradise Sponsors
+ required: false
Chest:
points: 1
required: false
- Tail:
- points: 1 # Lost Paradise Sponsors
- required: falseLikely invalid or redundant comment.
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_fox_parts.yml
Show resolved
Hide resolved
Resources/Prototypes/_LostParadise/Entities/Mobs/Customization/slime_cat_parts.yml
Show resolved
Hide resolved
Signed-off-by: Remuchi <[email protected]>
Signed-off-by: Remuchi <[email protected]>
|
В этом ПР'е есть конфликты, пожалуйста, устраните их, прежде чем мы сможем оценить ваш ПР. |
|
Реши конфликты слияния. |
Описание обновления:
Ported space-syndicate/space-station-14-next@d473150
Also added elf ears to slime people by @Farrellka-dev request.
Also fixed your RSI validator since it was throwing errors constantly.
Медиа
Проверки
Изменения
🆑