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

Centcom Tactical Bat #671

Merged
merged 18 commits into from
Nov 13, 2023
Merged

Centcom Tactical Bat #671

merged 18 commits into from
Nov 13, 2023

Conversation

Fullonibus
Copy link

@Fullonibus Fullonibus commented Nov 10, 2023

Что этот PR делает

Возвращает тактическую бейсбольную биту в игру.

Почему это хорошо для игры

Ультимативное оружие правосудия вновь будет нести свою службу на благо Нанотрейзен. Бойтесь, колени!

Изображения изменений

image

Тестирование

Функционал проверен в игре: иконки рабочие, куклы не с цк в руки не возьмут, складывается, раскладывается, хоумраны прописывает.

Changelog

🆑
add: Тактическая бита центрального командования.
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Nov 10, 2023
Copy link
Collaborator

@larentoun larentoun left a comment

Choose a reason for hiding this comment

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

Такое ощущение, что следовало бы наследоваться от телескопички

@dj-34
Copy link
Collaborator

dj-34 commented Nov 10, 2023

Линтер поправь

@Fullonibus
Copy link
Author

Такое ощущение, что следовало бы наследоваться от телескопички

тогда бы пришлось прописывать хоумран. А у биты уже он есть

modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
add comment
@larentoun
Copy link
Collaborator

Такое ощущение, что следовало бы наследоваться от телескопички

тогда бы пришлось прописывать хоумран. А у биты уже он есть

У тебя он все равно нету хоумрана, так как не вызываешь родителя. Предлагаю просто наследоваться от телескопички, и при ударе делать взрыв и толкать, если телескопичка открыта

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

вроде норм

@Fullonibus
Copy link
Author

Убрал ненужный код в инициализации, лишний ретурн, поправил иф-елс блок, добавил проверку в атаке.

@Fullonibus Fullonibus requested a review from larentoun November 11, 2023 14:04
Copy link
Collaborator

@larentoun larentoun left a comment

Choose a reason for hiding this comment

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

Ты в игре точно проверил, что оно работает как бита, а не телескопичка?

И хомраны, и дефлекты, и подготовки к хомрану?

modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
@Fullonibus
Copy link
Author

Ты в игре точно проверил, что оно работает как бита, а не телескопичка?

И хомраны, и дефлекты, и подготовки к хомрану?

Хоумраны работают
image
Зарядку убрал, она на бите ни к чему. Режим дефлекта не дает делать удары, поэтому его не ставил.

modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
w_class = WEIGHT_CLASS_HUGE //doesnt fit in backpack when its on for balance
force = force_on
attack_verb = attack_verb_on
homerun_ready = TRUE
Copy link
Collaborator

Choose a reason for hiding this comment

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

homerun_ready нет смысла выставлять, так как оно всегда включено при атаке, разве не так?

modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
@Fullonibus Fullonibus requested a review from larentoun November 13, 2023 13:31
Copy link
Collaborator

@larentoun larentoun left a comment

Choose a reason for hiding this comment

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

Если работает в игре, то сойдет

Copy link
Collaborator

@Gaxeer Gaxeer left a comment

Choose a reason for hiding this comment

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

На производительность не повлияет, чуток говнокода в модуле похуй. Как добавили, так и снесем, если надо будет.

@AyIong AyIong merged commit 4f08d76 into ss220club:master Nov 13, 2023
13 checks passed
@Fullonibus Fullonibus deleted the centcom branch November 13, 2023 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants