-
Notifications
You must be signed in to change notification settings - Fork 154
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
[Port] Heretics #1048
base: master
Are you sure you want to change the base?
[Port] Heretics #1048
Conversation
<!-- Что вы изменили в этом пулл реквесте? --> еретиики, взято с Goob-Station/Goob-Station#563 Goob-Station/Goob-Station#628 и Goob-Station/Goob-Station#615 <!-- Почему оно было изменено? Ссылайтесь на любые обсуждения или вопросы здесь. Пожалуйста, обсудите, как это повлияет на игровой баланс. --> перенос с сс13 <!-- Если речь идет об изменении кода, кратко изложите на высоком уровне принцип работы нового кода. Это облегчает рецензирование.- --> ничего глобально не поменяли, вроде <!-- Пулл реквесты, которые вносят внутриигровые изменения (добавление одежды, предметов, новых возможностей и т.д.), должны содержать медиа, демонстрирующие изменения. Небольшие исправления/рефакторы не требуют медиа. Если Вы не уверены в том, что Ваш пулл реквест требует медиа, спросите мейнтейнера. --> <!-- В связи с наплывом ПР'ов нам необходимо убедиться, что ПР'ы следуют правильным рекомендациям. Пожалуйста, уделите время прочтению, если делаете пулл реквест (ПР) впервые. Отметьте поля ниже, чтобы подтвердить, что Вы действительно видели их (поставьте X в скобках, например [X]): --> - [x] Я прочитал(а) и следую [Руководство по созданию пулл реквестов](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html). Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера. - [x] Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, **или** этот пулл реквест не требует демонстрации в игре <!-- Перечислите все критические изменения, включая изменения пространства имён, публичных классов/методов/полей, переименования прототипов, и предоставьте инструкции по их исправлению. --> **Чейнджлог** <!-- Здесь Вы можете заполнить журнал изменений, который будет автоматически добавлен в игру при мердже Вашего пулл реквест. Чтобы игроки узнали о новых возможностях и изменениях, которые могут повлиять на их игру, добавьте запись в журнал изменений. Не считайте суффикс типа записи (например, add) "частью" предложения: плохо: - add: новый инструмент для инженеров хорошо: - add: добавлен новый инструмент для инженеров Помещение имени после символа 🆑 изменит имя, которое будет отображаться в журнале изменений (в противном случае будет использоваться ваше имя пользователя GitHub). Например: 🆑 AruMoon --> 🆑 Ratyyy - add: На некоторых станциях начали возникать разломы реальности, а некоторые люди интересоваться ими...
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 59 files out of 166 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the 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 (
|
RSI Diff Bot; head commit e301b60 merging into 247dd18 Resources/Textures/ADT/Heretic/Blades/blade_ash-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_blade-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_cosmos-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_flesh-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_lock-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_moon-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_rust-inhand.rsi
Resources/Textures/ADT/Heretic/Blades/blade_void-inhand.rsi
Resources/Textures/ADT/Heretic/abilities_heretic.rsi
Resources/Textures/ADT/Heretic/combat_marks.rsi
Resources/Textures/ADT/Heretic/eldritch_armor.rsi
Resources/Textures/ADT/Heretic/eldritch_hood.rsi
Resources/Textures/ADT/Heretic/eldritch_mobs.rsi
Resources/Textures/ADT/Heretic/mansus_grasp.rsi
Resources/Textures/ADT/Heretic/polymorphs.rsi
Resources/Textures/ADT/Heretic/reality_fracture.rsi
Resources/Textures/ADT/Heretic/ritual_rune.rsi
|
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.
нужен рефактор
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Спасибо, обязательно возьмём 🌞 |
В общем, я устал рефакторить этот чудо кусок кода. Пр не будет одобрен пока не будет сделан адекватный рефакторинг |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
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.
Нужно тестирование, что рефактор не сломал
Описание PR
добавляет еретиков реально
Полностью пофикшены и работают
🆑 Rouden