-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Fix] Some Body Type Bugs #276
base: master
Are you sure you want to change the base?
Conversation
WalkthroughВнесены изменения в YML-файлы, определяющие прототипы мобов. В одном файле для рептилий упрощены ключи смещений у женских персонажей. В гарпийском файле добавлено ограничение по полу и новые базовые спрайты для мужских вариантов. Кроме того, в файлах для человеческих и рептилий (папка Changes
Suggested labels
Suggested reviewers
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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (6)
Resources/Prototypes/_White/Species/reptilian.yml (2)
34-39
: Новый спрайт для мужской головы
Новый элементMobReptilianSlimHeadMale
добавлен для отображения мужской версии головы. Он использует те же базовые значения (sprite и state), что и женский вариант, что соответствует требованиям для единообразия.
Обратите внимание: в строке 39 обнаружены завершающие пробелы — рекомендуется удалить их для соблюдения чистоты кода.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 39-39: trailing spaces
(trailing-spaces)
52-57
: Новый спрайт для мужского торса
Добавлен элементMobReptilianSlimTorsoMale
для представления мужской версии торса, с параметрами, аналогичными женскому варианту. Если для мужской версии требуется иное значениеstate
, проверьте корректность текущего значения.
Также удалите завершающие пробелы в строке 57 для исправления мелкой стилистической проблемы.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 57-57: trailing spaces
(trailing-spaces)
Resources/Prototypes/_White/Species/human.yml (2)
34-39
: Новый спрайт для мужской головы (Human)
Добавлен элементMobHumanSlimHeadMale
для мужской вариации головы. Значениеstate: head_slim_f
оставлено таким же, как у женской версии – убедитесь, что это соответствует игровым требованиям.
Также обратите внимание на завершающие пробелы в строке 39, которые следует удалить.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 39-39: trailing spaces
(trailing-spaces)
52-57
: Новый спрайт для мужского торса (Human)
Добавлен элементMobHumanSlimTorsoMale
для мужского варианта торса с аналогичной базовой настройкой. Проверьте, корректен ли используемыйstate: torso_slim_f
для мужской версии, или требуется иное значение.
Не забудьте удалить завершающие пробелы в строке 57.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 57-57: trailing spaces
(trailing-spaces)
Resources/Prototypes/Entities/Mobs/Species/reptilian.yml (2)
88-98
: Упрощение ключей femaleDisplacements
В блоке конфигурацииfemaleDisplacements
для инвентаря изменены ключи: теперь для перемещений используются сокращённые идентификаторыjumpsuit
иshoes
вместо прежнихjumpsuit-body-digitigrade
иshoes-body-digitigrade
. Это упрощение должно сделать конфигурацию более понятной и единообразной.
157-168
: Консистентное обновление для Dummy-сущности
Аналогичные изменения ключей внесены в блокfemaleDisplacements
для Dummy-сущности (MobReptilianDummy
). Новые ключиjumpsuit
иshoes
используют те же настройки, что и в основном блоке. Это обеспечивает единообразие в определении перемещений у разных сущностей.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
Resources/Prototypes/Entities/Mobs/Species/reptilian.yml
(2 hunks)Resources/Prototypes/_White/Species/harpy.yml
(3 hunks)Resources/Prototypes/_White/Species/human.yml
(2 hunks)Resources/Prototypes/_White/Species/reptilian.yml
(2 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/_White/Species/reptilian.yml
[error] 39-39: trailing spaces
(trailing-spaces)
[error] 57-57: trailing spaces
(trailing-spaces)
Resources/Prototypes/_White/Species/human.yml
[error] 39-39: trailing spaces
(trailing-spaces)
[error] 57-57: trailing spaces
(trailing-spaces)
⏰ Context from checks skipped due to timeout of 90000ms (5)
- GitHub Check: YAML map schema validator
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: YAML Linter
🔇 Additional comments (3)
Resources/Prototypes/_White/Species/harpy.yml (3)
4-5
: Добавлено ограничение по полу для HarpySlim
Добавлено новое полеsexRestrictions
с ограничением наMale
для типа телаHarpySlim
. Это изменение должно обеспечить, чтобы данный тип использовался только для мужских персонажей.
32-37
: Новый спрайт для мужской головы у гарпий
Введён элементMobHarpyHeadSlimMale
для мужской версии головы с использованием тех же базовых параметров, что и у женской. Проверьте, соответствует ли значениеstate: head_slim_f
заданному стилю отображения для мужских персонажей.
50-55
: Новый спрайт для мужского торса у гарпий
Добавлен новый элементMobHarpyTorsoSlimMale
для мужской версии торса. Параметры базового спрайта идентичны женскому варианту – убедитесь, что это осознанное решение.
Описание PR
Я дурачек
Изменения
🆑