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] Adds buckshot shuffler #1024

Merged
merged 8 commits into from
Oct 6, 2024

Conversation

MrRomainzZ
Copy link

@MrRomainzZ MrRomainzZ commented Oct 4, 2024

ПР добавляет пермешиватель бакшотов. Работает как коробка патронов, но с возможностью перемешивать пули при нажатии на пробел (кнопка по умолчанию).

Перемешиватель можно распечатать на автолате за 4 листа железа во вкладке "Безопасность". Им можно собирать использованные и заряженные бакшоты.

Позволяет играть в "Buckshot Roulette" без участия третьего человека.

Видео с изменениями (В видео была допущена ошибка. По правилам игры выше, после пустого выстрела в себя ты продолжаешь ход)
BuckshotShufflerPR.mp4

  • Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
  • Я внимательно прочитал все свои изменения и багов в них не нашел.
  • Я запускал сервер со своими изменениями локально и все протестировал.

@github-actions github-actions bot added 🎨 Спрайты 🎨 Есть спрайты 🧱 Код 🧱 Изменения .dm файлов 🔵 Требуется проверка 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! labels Oct 4, 2024
Copy link
Collaborator

@RalseiDreemuurr RalseiDreemuurr left a comment

Choose a reason for hiding this comment

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

Нужно изменение, к которому оставлен коммент. Плюсом я не понимаю, для чего это сделано. Не помню предложки на это

@RalseiDreemuurr RalseiDreemuurr added ❗ PR НЕ МЁРЖИТЬ ❗ PR НЕ МЁРЖИТЬ 🔵 Требует рассмотрения PR скорее всего не относится к таскам разработки. Будет проверка labels Oct 4, 2024
@MrRomainzZ
Copy link
Author

Предложка была в виде друга в дискорде. И не знаю, что плохого в "штучке"

@RalseiDreemuurr
Copy link
Collaborator

Предложка была в виде друга в дискорде. И не знаю, что плохого в "штучке"

Я не говорю, что плохо. Просто у нас ПР-ы, не проходящие предложки или не находящиеся в тасках просто так не принимаются. В общем, решай вопрос с хэдкодером

RalseiDreemuurr
RalseiDreemuurr previously approved these changes Oct 4, 2024
Copy link

github-actions bot commented Oct 5, 2024

Pull Request содержит конфликты. Устраните их, чтобы мы смогли оценить и принять запрос.

@MrCat15352
Copy link

Конфликты + у тебя ещё

@github-actions github-actions bot removed the 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! label Oct 6, 2024
@MrRomainzZ MrRomainzZ requested a review from MrCat15352 October 6, 2024 07:43
@MrCat15352 MrCat15352 removed the ❗ PR НЕ МЁРЖИТЬ ❗ PR НЕ МЁРЖИТЬ label Oct 6, 2024
@MrCat15352 MrCat15352 merged commit aed67d6 into CeladonSS13:beta-dev Oct 6, 2024
3 checks passed
@MrRomainzZ MrRomainzZ deleted the buckshot-shuffler branch October 6, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Спрайты 🎨 Есть спрайты 🔵 Требует рассмотрения PR скорее всего не относится к таскам разработки. Будет проверка 🔵 Требуется проверка 🧱 Код 🧱 Изменения .dm файлов
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants