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
@ss220bot ss220bot added the 📜 CL невалиден Этот чейнджлог не пройдет валидацию перед публикацией. Исправить или удалить, если не требуется label Dec 23, 2024
@Drsmail
Copy link
Author

Drsmail commented Dec 23, 2024

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

@github-actions github-actions bot removed the Merge Conflict Ну блять... label Dec 23, 2024
@ss220bot ss220bot added 📜 CL валиден Этот чейнджлог будет успешно опубликован and removed 📜 CL невалиден Этот чейнджлог не пройдет валидацию перед публикацией. Исправить или удалить, если не требуется labels Dec 23, 2024
@m-dzianishchyts m-dzianishchyts self-requested a review December 23, 2024 22:57
@ss220bot ss220bot added 📜 CL невалиден Этот чейнджлог не пройдет валидацию перед публикацией. Исправить или удалить, если не требуется and removed 📜 CL валиден Этот чейнджлог будет успешно опубликован labels 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
@ss220app ss220app bot added 📜 CL валиден Этот чейнджлог будет успешно опубликован and removed 📜 CL невалиден Этот чейнджлог не пройдет валидацию перед публикацией. Исправить или удалить, если не требуется labels 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

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

@Drsmail
Copy link
Author

Drsmail commented Jan 19, 2025

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

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

А ещё мне надо как-то настроить почту, потому что мне по 50 писем гит присылает из-за того что я ревью иногда делаю на офах. Я вот пропустил важное сообщение от Максимилиана.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💾 Изменение конфига Ф-ф-фуриор...? 📜 CL валиден Этот чейнджлог будет успешно опубликован Testmerge Required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants