hackhitchin / piwars3

Piwars 3.0
2 stars 2 forks source link

Investigate use of JSON for config files #11

Open robberwick opened 7 years ago

robberwick commented 7 years ago

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 commented 7 years ago

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 commented 7 years ago

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.