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

Investigate use of JSON for config files #11

Open
robberwick opened this issue Mar 19, 2017 · 2 comments
Open

Investigate use of JSON for config files #11

robberwick opened this issue Mar 19, 2017 · 2 comments
Assignees

Comments

@robberwick
Copy link

Currently, we're using INI files in order to persist configuration parameters in a file, which is quite verbose when serialising and deserialising. If we switched to using JSON, then we would be able to directly save and load a single python configuration dictionary by using json.dumps(conf_obj) and json.loads(file_content).

@worlds6440
Copy link
Contributor

I agree, INI file can be a little verbose in code for reading/writing. I will look into JSON files in a bit when I have more challenges written.

@robberwick
Copy link
Author

INI files are perfectly fine in the short term, so there's no real gain in switching to JSON in preference to writing challenges. It's certainly not vital for the competition. I'm happy to look at this as a nice to have feature if you assign it to me.

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

No branches or pull requests

2 participants