Skip to content

Simplify and centralize configuration loading mechanisms - #872

Merged
jvyden merged 3 commits into
LittleBigRefresh:mainfrom
jvyden:config-refactor
Jul 22, 2025
Merged

Simplify and centralize configuration loading mechanisms#872
jvyden merged 3 commits into
LittleBigRefresh:mainfrom
jvyden:config-refactor

Conversation

@jvyden

@jvyden jvyden commented Jul 22, 2025

Copy link
Copy Markdown
Member
  • Allows tests to easily generate empty configs. Prevents them from using filesystem configs entirely.
  • Configuration loading is behind a lock now. This nullifies the weird random failures caused by unit tests generating config files but doesn't actually serve too much purpose as the codepath isn't hit by unit tests anymore. Kept as it shouldn't technically harm anything.
  • Reduces the complexity of config management in RefreshGameServer and TestRefreshGameServer
  • Allows the new Refresh.WorkerManager server to easily utilize all the configs from the main GameServer project

@jvyden
jvyden merged commit 2539cb1 into LittleBigRefresh:main Jul 22, 2025
@jvyden
jvyden deleted the config-refactor branch July 22, 2025 04:21
@jvyden jvyden added this to the Import dry archive milestone Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant