Skip to content
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

add/refactor: сentcomm rapier #5924

Merged
merged 20 commits into from
Oct 3, 2024

Conversation

dageavtobusnick
Copy link
Contributor

@dageavtobusnick dageavtobusnick commented Sep 19, 2024

Описание

Добавляет рапиру центрального командования.
Характеристики:
Урон 25, при броске 20
Пробитие брони 100
Урон стамине 100
Шанс блока 95 процентов. Блокирует все типы атак.
При разборе в деструктивном анализаторе дает техи, аналогичные синдирапире.
При разборе в автолете не дает ресурсов, аналогично синдирапире.
При ударе по цели в стаминокрите ложит ее в сон, аналогично синдирапире.
Имеет антидроп, который активируется на Z или кнопкой в левом верхнем углу экрана, при наличии рапиры в руках.
Прочие характеристики аналогичны обычной рапире и синдирапире.
Эта рапира в ножнах добавлена на пояс в аутфиты NT Navy Captain и Supreme Commander.
Спрайты рапиры нарисовал Кисик @splitpussy.
Так же в этом пр-е отрефакторены блоки у оружия. Теперь чтобы исключить урон из блока не нужно переопределять hit_reaction. Достаточно просто убрать битмаску атаки из поля block_type. Так же вынес стаминоурон и сон при стаминокрите при успешной атаке в отдельный компонент. Помимо этого, отрефакторены рапира и синдирапира, с вынесением общих свойств в отдельные классы, во избежание копирования. А так же мелкие рефакторы, связанные с неймингом флагов атаки.

Ссылка на предложение/Причина создания ПР

Рефактор + щитспавн игрушка для админов. Предложка не нужна.

Демонстрация изменений

image

Тесты

Запустил, протестил механики рапиры и отображение спрайтов, протестил механику блока на типовом оружии, которое подверглось рефактору. Все работает без багов.

@github-actions github-actions bot added the DMI Affects DMI files label Sep 19, 2024
@dageavtobusnick dageavtobusnick changed the title add/refactor: centcomm rapier add/refactor: сentcomm rapier Sep 19, 2024
@dageavtobusnick
Copy link
Contributor Author

dageavtobusnick commented Sep 19, 2024

Гребанное автозакрытие пр-а...

code/datums/action.dm Outdated Show resolved Hide resolved
@Rafnik02
Copy link
Contributor

Синдирапира не рапира, а сабля.

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Sep 22, 2024
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Sep 23, 2024
@SoulFN
Copy link

SoulFN commented Sep 24, 2024

Надо на тандердом. Зачем? А потому что.

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Sep 27, 2024
@dageavtobusnick
Copy link
Contributor Author

Ну началось...

@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Sep 27, 2024
@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Oct 1, 2024
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Oct 1, 2024
Copy link
Member

@Daeberdir Daeberdir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Впринципе, пойдёт. Но всё равно нужен ещё один полноценный рефактор цепочки атак, чтобы перейти на такую систему.

@Daeberdir Daeberdir added the Has Approve Have been approved by somebody from Review Team and awaiting for one more approve label Oct 3, 2024
Copy link
Contributor

@Den109G Den109G left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хороший рефактор. Работает всё.

@Den109G Den109G added this pull request to the merge queue Oct 3, 2024
Merged via the queue into ss220-space:master220 with commit 7f5ce4c Oct 3, 2024
12 checks passed
@dageavtobusnick dageavtobusnick deleted the сentcom_rapier branch October 4, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files Has Approve Have been approved by somebody from Review Team and awaiting for one more approve
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants