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

[Return] brigmedic #2202

Closed
wants to merge 29 commits into from
Closed

[Return] brigmedic #2202

wants to merge 29 commits into from

Conversation

CatBackGround
Copy link
Contributor

@CatBackGround CatBackGround commented Jun 3, 2024

Описание PR

Итак, я возвращаю бригмедика в строй.

Почему?

Во-первых, это интересная роль как со стороны геймплея, так и со стороны РП-отырыша.

Почему PR был отправлен именно в Corvax?

Почитав официальный Дискорд сервер SS и PRы на основном репозитории по поводу Бригмедика, я смог вывести мысль о том, что Визардам пока что никуда не усрался Бригмедик.

Что будет, если и Corvax'у не нужен будет БМ и PR отклонят?

Осознаю всю степень несправедливости в политике Corvax, и уйду в закат.

Недоделанные аспекты

HiddenDescription пока что не поддерживает бригмедика. (В ПРОЦЕССЕ)

Медиа

image

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 CatBG

  • add: Бригмедик возвращается в строй!
  • tweak: Изменена начальная экипировка бригмедика.

@CatBackGround CatBackGround requested a review from Morb0 as a code owner June 3, 2024 13:33
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: No C# Code has not been changed Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation labels Jun 3, 2024
@lzk228
Copy link
Collaborator

lzk228 commented Jun 3, 2024

не

@lzk228 lzk228 closed this Jun 3, 2024
@lapatison
Copy link
Contributor

Реально.
Неоднократно же обсуждали. Зачем делать ПР на тему, которую давно решили не добавлять?

@Hemomancer
Copy link

Пилота добавили.

@lzk228
Copy link
Collaborator

lzk228 commented Jun 3, 2024

на тот момент я просто ещё не мог его отклонить

@Hemomancer
Copy link

Получается, чудеса случаются.

@poeMota
Copy link
Contributor

poeMota commented Jun 3, 2024

на тот момент я просто ещё не мог его отклонить

тебе права дали чтоб ты хуйнёй занимался и просто ПРы закрывал?

@foboscheshir
Copy link
Contributor

бригмед не нужон

@CatBackGround CatBackGround mentioned this pull request Jun 7, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation Changes: No C# Code has not been changed Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants