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

Antag mix Whitelist #1710

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Conversation

Drsmail
Copy link

@Drsmail Drsmail commented Dec 23, 2024

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

Добавляет белый список рас в сценарии антагов для Antaga_mix.
Немного модифицирует логику отбора кандидатов.
Исправляет ошибку, из-за которой игроки с куклами воксов, получали случайное имя.

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

Не нужно будет обновлять конфиг, при добавление новой расы.

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

Проверил на локалхосте.
Заспавнился за агента синдиката
Заспавнился за Пожирателя разума.

Changelog

🆑
fix: При наборе в команду воксов рейдеров, для спавана воксов будет использована кукла игрока, если есть.
/:cl:

@github-actions github-actions bot added Merge Conflict Ну блять... 💾 Изменение конфига Ф-ф-фуриор...? labels Dec 23, 2024
@Drsmail Drsmail marked this pull request as ready for review December 23, 2024 22:29
@Drsmail
Copy link
Author

Drsmail commented Dec 23, 2024

А что надо писать, если чейнджлога нет? Я просто его удалил весь.

@Drsmail
Copy link
Author

Drsmail commented Dec 24, 2024

@m-dzianishchyts, пупс мне говорит, что нужен тест мёрге и протестировать это на сервере, с форсам раунда воксов, чтобы убедится, что куклы воксов игроков, корректно спавнятся. Надо наверное для этого добавить логии?

@m-dzianishchyts
Copy link
Collaborator

Обнови чейнджлог еще

Co-authored-by: Mikhail Dzianishchyts <[email protected]>
@PhantornRU
Copy link
Collaborator

@m-dzianishchyts, пупс мне говорит, что нужен тест мёрге и протестировать это на сервере, с форсам раунда воксов, чтобы убедится, что куклы воксов игроков, корректно спавнятся. Надо наверное для этого добавить логии?

на время тест мердже можно добавить логи чтобы потом их вытащить их дебаг логиса.
Главное не забудь их потом убрать (можешь для себя это даже как-то выделить чтобы в случае чего не забыть убрать)

@PhantornRU PhantornRU self-assigned this Dec 26, 2024
@Drsmail
Copy link
Author

Drsmail commented Dec 26, 2024

@m-dzianishchyts, пупс мне говорит, что нужен тест мёрге и протестировать это на сервере, с форсам раунда воксов, чтобы убедится, что куклы воксов игроков, корректно спавнятся. Надо наверное для этого добавить логии?

на время тест мердже можно добавить логи чтобы потом их вытащить их дебаг логиса. Главное не забудь их потом убрать (можешь для себя это даже как-то выделить чтобы в случае чего не забыть убрать)

Да, я как раз хотел этим сейчас заняться. Не подскажешь, где можно пример глянуть?

@PhantornRU
Copy link
Collaborator

PhantornRU commented Dec 26, 2024

@m-dzianishchyts, пупс мне говорит, что нужен тест мёрге и протестировать это на сервере, с форсам раунда воксов, чтобы убедится, что куклы воксов игроков, корректно спавнятся. Надо наверное для этого добавить логии?

на время тест мердже можно добавить логи чтобы потом их вытащить их дебаг логиса. Главное не забудь их потом убрать (можешь для себя это даже как-то выделить чтобы в случае чего не забыть убрать)

Да, я как раз хотел этим сейчас заняться. Не подскажешь, где можно пример глянуть?

c3b639f
Потом вытягиваешь через логи (старые остановки)

@m-dzianishchyts
Copy link
Collaborator

Какая тут ситуация вообще?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants