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.

Sign up for GitHub

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

Jump to bottom

Decouple configs from application's state #422

Merged

aome510 merged 4 commits into master from separate-configs-from-state

Apr 21, 2024

Owner

aome510 commented Apr 21, 2024 •

edited

Changes

remove configs: Configs from application's State struct
add config::CONFIGS as a global static variable representing the application's configs
implement config::get_config to get a static reference of config::CONFIGS and config::set_config to set the global variable

aome510 added 4 commits

April 21, 2024 13:56


          move Configs from state module to config module

7f62101


          use config::get_config() to get the application's configs

8cc38e0


          remove redundant state: &SharedState as function's argument

ff2dd6f


          cleanup

905a239

aome510 merged commit d91318e into master

3 checks passed

aome510 deleted the separate-configs-from-state branch

April 21, 2024 18:57

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