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

no thief during nukie round fix #1926

Merged

Conversation

lexaSvarshik
Copy link

Описание PR

Пока что это изменение убирает только вора. Если позже добавят какие-то SubGameModes, их не будет тоже. Но будут ли они уместны в раунде с ЯО? Вряд ли.

PR по задаче

no SubGamemodeRule during nukie round

Медиа

Проверки

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

Изменения

🆑 Svarshik

  • fix: Вор вырезан из режима Ядерных Оперативников

пока что это убирает только вора. Если позже добавят какие-то подгеммоды, их не будет тоже. Но будут ли они нужны в раунде с ЯО? Вряд ли.
@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Sep 23, 2024
@Ady4ik
Copy link

Ady4ik commented Sep 24, 2024

Сабгейммод кроме вора ничего не контестит разве? @DexlerXD

@lexaSvarshik
Copy link
Author

Сабгейммод кроме вора ничего не контестит разве? @DexlerXD

Судя по тому, что я нашёл, вор - единственный субгейммод. Вот ПР от оффов, где его таковым сделали

Ну и вот оффы пилят ещё какой-то субгейммод, но он в том числе не нужен на нюке

@DexlerXD
Copy link

ничего против не имею

@DexlerXD
Copy link

@Ady4ik

@Ady4ik Ady4ik merged commit 0eeb1fa into SerbiaStrong-220:master Sep 25, 2024
14 checks passed
@lexaSvarshik lexaSvarshik deleted the no-thief-on-nukie-round-fix branch October 9, 2024 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants