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

Fix return value of pool spawner initialization #1764

Merged

Conversation

m-dzianishchyts
Copy link
Collaborator

@m-dzianishchyts m-dzianishchyts commented Jan 24, 2025

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

Чинит спавн лута в космосе.

Changelog

🆑 Maxiemar
fix: Исправлен спавн лута в космосе.
/:cl:

Summary by Sourcery

Fix space loot spawning and increase the available points for loot generation to account for additional usage in centcomm and the gateway.

Bug Fixes:

  • Fixed a bug that prevented space loot from spawning correctly.

Enhancements:

  • Increased the available points for loot generation.

@github-actions github-actions bot added 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. :shipit: Улучшение Наконец-то это сделали удобнее и лучше... labels Jan 24, 2025
@ss220app ss220app bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Jan 24, 2025
@kyunkyunkyun
Copy link

у нас ЦК и гейты же пока не фигурируют никак в пуле спавна лута, рановато повышается бюджет, нет?

@m-dzianishchyts
Copy link
Collaborator Author

m-dzianishchyts commented Jan 24, 2025

у нас ЦК и гейты же пока не фигурируют никак в пуле спавна лута, рановато повышается бюджет, нет?

image

@kyunkyunkyun
Copy link

image

уээ

@m-dzianishchyts m-dzianishchyts changed the title Spaceloot fix tweak Spaceloot changes Jan 24, 2025
@m-dzianishchyts m-dzianishchyts changed the title Spaceloot changes [TM] Spaceloot changes Jan 24, 2025
Copy link

github-actions bot commented Feb 8, 2025

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Feb 8, 2025
@m-dzianishchyts m-dzianishchyts added Stale Exempt PR не может быть устаревшим. Видать кто-то подкупил хэда. and removed Stale ПР долго был не активен, и требует обновления. labels Feb 8, 2025
m-dzianishchyts pushed a commit that referenced this pull request Feb 11, 2025
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

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

Добавляет оффовские спавнеры лута в модульные руины.
Добавляет несколько новых, модульных спавнеров для определённых руин.
Добавляет тиреи на те наши руины, где присутствуют сейфы с лутом, чтобы
это было более очевидно для гейтеров и им не приходилось лезть в код
чтобы найти заветный сейф, который иначе никак не обнаружить.
Слегка передвигает одного из воксов на руине `voxraiders_1` и добавляет
ему стеклянный аирлок, чтобы он не игнорировал лутающегося рядом
гейтера.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes #xxxx" (где xxxx - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

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

Больше рандома = больше реиграбельности.

Пока без гейтов, но если всё норм, то гейты тоже можно будет зааджастить
позднее, как уже частично произошло со спейсбаттлом.

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

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

Поюзал все спавнеры на локалке, без проблем, вроде. Наверное, стоит
юзать в ТМе, пока не замержат #1764

<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
tweak: Модульные руины обновлены в соответствии с недавним ребалансом
лута космоса с оффов.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->

## Summary by Sourcery

Update modular space ruins with the new loot system.

New Features:
- Added off-station loot spawners to modular space ruins.
- Added new modular spawners for specific ruins.

Tests:
- Tested all spawners locally.
@github-actions github-actions bot added the Merge Conflict Ну блять... label Feb 11, 2025
@github-actions github-actions bot added Merge Conflict Ну блять... :finnadie: НЕ МОДУЛЬНО Ну это ваще пиздец. Либо засовывай в модуль, либо заливай на апстрим. and removed Merge Conflict Ну блять... :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. labels Feb 11, 2025
@m-dzianishchyts m-dzianishchyts changed the title [TM] Spaceloot changes Fix return value of pool spawner initialization Feb 11, 2025
@ss220club ss220club deleted a comment from sourcery-ai bot Feb 11, 2025
@m-dzianishchyts m-dzianishchyts added the 🕛 Ждёт мержа Совсем скоро это будет залито label Feb 11, 2025
@github-actions github-actions bot removed the Merge Conflict Ну блять... label Feb 11, 2025
@m-dzianishchyts m-dzianishchyts merged commit ae6457a into ss220club:master Feb 18, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕛 Ждёт мержа Совсем скоро это будет залито :finnadie: НЕ МОДУЛЬНО Ну это ваще пиздец. Либо засовывай в модуль, либо заливай на апстрим. 📜 CL валиден Этот чейнджлог будет успешно опубликован :shipit: Улучшение Наконец-то это сделали удобнее и лучше... Stale Exempt PR не может быть устаревшим. Видать кто-то подкупил хэда. Testmerge Required 🔧 Фикс Переписываем ошибку так, чтобы она проявлялась в других обстоятельствах
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants