-
Notifications
You must be signed in to change notification settings - Fork 64
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
Melee rework 1/4: combo system #1108
Conversation
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.
Опять я со своим ненужным мнением. Здравствуйте
Аой |
Ну гитхаб, ну шутник. |
Пож не забывайте про локаль, а то так копится и остальным разрабам приходится выискивать и фиксить |
Content.Server/ADT/AddComponentsOnUse/Systems/AddComponentsOnUseSystem.cs
Show resolved
Hide resolved
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.
РЕВЬЮ ЛОКАЛИ
- Забыты запятые перед "обучающий" в двух местах (причастный оборот) и перед "который обучает" (сложноподчинённое предложение)
- Также, аббревиатуры (CQC) обычно пишутся заглавными буквами. В названии предмета можно и большими, если остальные слова с маленькой буквы
- Запятая перед союзом "и" в "и приобретается отдельно" не нужна, так как придаточные части "не входит" и " приобретается" относятся к одной основной части "взгляд на тысячу ярдов"
Описание PR
добавила систему комбо ударов, дальше читать чейнджлог
Почему / Баланс
Техническая информация
все comboEffects в компоненте потому, что там удобнее
так же в методе для кнокдауна 2 новых поля down(упадёт ли человек после стана) и dropitems, что отвечает за выпадение вещей из рук после падения
Медиа
я бы показала, но весит слишком много
Требования
Критические изменения
Чейнджлог
🆑 Ratyyy & Котя & Тамиоки