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

изменение промежутка между спавном метеоров #2309

Merged
merged 2 commits into from
Jun 23, 2024
Merged

изменение промежутка между спавном метеоров #2309

merged 2 commits into from
Jun 23, 2024

Conversation

comasqw
Copy link
Contributor

@comasqw comasqw commented Jun 20, 2024

Описание PR

Изменяет спавн-рейт метеоров с 7.5 - 12.5 до 30 - 45 минут.
Предложка - https://discord.com/channels/919301044784226385/1253333510928863323

Теперь за двух часовую смену ивент с метеорами может произойти 2-4 раза,
а не как раньше 12+раз. Даже учитывая данный ПР - space-wizards/space-station-14#29194,
количество этих ивентов остаётся в районе 10, что ситуацию никак не меняет.

Медиа

Проверки

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

Изменения

🆑

  • tweak: Изменён промежуток между спавном метеоров с 7.5 - 12.5 до 30 - 45 минут

@comasqw comasqw requested a review from Morb0 as a code owner June 20, 2024 17:43
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label Jun 20, 2024
Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

конфигурация на то и конфигурация, чтобы менять в конфигурации сервера
перенести в Resources/ConfigPresets/Corvax/common.toml

@github-actions github-actions bot added the Changes: No C# Code has not been changed label Jun 20, 2024
Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

одобряю реализацию, но не саму идею, но так же и не против идеи, я нейтральный челик

@comasqw
Copy link
Contributor Author

comasqw commented Jun 20, 2024

тест упал рандомно, из-за файла локализации -_-
image

@Morb0 Morb0 merged commit 11155ac into space-syndicate:master Jun 23, 2024
12 checks passed
@comasqw comasqw deleted the change_meteors_spawn_rate branch June 23, 2024 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

3 participants