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: Выкидной ножик aka стилет #994

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

HardexZ
Copy link

@HardexZ HardexZ commented Feb 6, 2024

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

Добавлен новый ножик который появляется в техах, шанс появления равен дымовой бомбе.
Наносит 8 брут урона, в сложенном состоянии имеет размер tiny, в разложенном small, так же его можно повесить на пояс.

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

Новое интересное оружие из техов.

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

image (1)
image
image3

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

Тест на локалке

Changelog

🆑
add: Добавлен выкидной нож в пул предметов контрабанды из техов

/:cl:

@github-actions github-actions bot added 🔉 Звук Услада для ушей. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! labels Feb 6, 2024
modular_ss220/objects/code/weapons.dm Outdated Show resolved Hide resolved
modular_ss220/objects/code/weapons.dm Show resolved Hide resolved
@dj-34 dj-34 added the :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. label Feb 7, 2024
@PhantornRU
Copy link
Collaborator

Я когда-то рассказывал как апдейтить стейт правильно. Пишу на примере "как апдейтить сакуру":

Пишешь прок для своего объекта и уже в нём как в примере:

"твой объект"/update_overlays()
  . = ..()
  cut_overlays()
  if(grass)
    add_overlays("[название добавляемого стейта травы]")
  add_overlays("[название стейта твоего дерева]")
  if(leaves)
    add_overlays("[название добавляемого стейта падающей анимированной листвы]")

Потом при взаимодействиях с этим деревом твоей тяпкой ты вызываешь прок:

  update_icon(UPDATE_OVERLAYS)

Примеры:
image

После чего ты должен где-то вызвать:
image

Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Попробуй изменить апдейты чтобы тебе не пришлось с ними ебаться

@github-actions github-actions bot added the Merge Conflict Ну блять... label Feb 9, 2024
@PhantornRU
Copy link
Collaborator

мерж конфликты

@AyIong
Copy link
Collaborator

AyIong commented Feb 26, 2024

3 недели и ни единого телодвижения, так дела не делаются, сделай запрошенные изменения в течении недели, или закрой ПР, иначе это сделаем мы

@HardexZ
Copy link
Author

HardexZ commented Feb 26, 2024

3 недели и ни единого телодвижения, так дела не делаются, сделай запрошенные изменения в течении недели, или закрой ПР, иначе это сделаем мы

ирл дела навалились на этой доделаю

Починил конфлиты, остались проки
@HardexZ HardexZ reopened this Feb 28, 2024
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Feb 28, 2024
@AyIong AyIong requested review from PhantornRU and Legendaxe March 12, 2024 18:14
@AyIong
Copy link
Collaborator

AyIong commented Mar 12, 2024

Если это будет висеть ещё 3 дня, я ебанусь
Ну-ка ревью давайте, а то запросили изменения и забыли :madge:

@Legendaxe
Copy link

Если это будет висеть ещё 3 дня, я ебанусь Ну-ка ревью давайте, а то запросили изменения и забыли :madge:

Так он не вынес изменения спрайта в update_icon_state

@AyIong
Copy link
Collaborator

AyIong commented Mar 12, 2024

@HardexZ слышал? Ну короче, если вообще никак не получится это сделать, можешь покнуть кого-то из разрабов в #разработчик-и-игрок
Или как там этот канал называется в дискорде

@HardexZ
Copy link
Author

HardexZ commented Mar 12, 2024

вроде исправил как должно быть

@HardexZ HardexZ requested a review from AyIong March 12, 2024 20:09
Copy link

@Legendaxe Legendaxe 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
Co-authored-by: Volodymir Ohorodnytskyi <[email protected]>
@Legendaxe Legendaxe merged commit 2b728d4 into ss220club:master Mar 12, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден 🔉 Звук Услада для ушей. :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants