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

Simplify Config loading #64

Open
bfops opened this issue Dec 18, 2010 · 2 comments
Open

Simplify Config loading #64

bfops opened this issue Dec 18, 2010 · 2 comments

Comments

@bfops
Copy link
Owner

bfops commented Dec 18, 2010

Config loading right now is error-prone and finicky. Changing parts requires changes in multiple locations. Fix

@bfops
Copy link
Owner Author

bfops commented Dec 24, 2010

Config broken into separate pieces in 01e32bb (merged in 711711f)

@bfops
Copy link
Owner Author

bfops commented Dec 24, 2010

Merging Config and ConfigLoader would be the next step, and avoid prefetching all the values. But then we would want to cache any already-fetched values, and we would end up with this setup again

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

No branches or pull requests

1 participant