Closed mattpalermo closed 7 years ago
Going to integrate this into dev and then integrate dev into master without waiting for review. I am a bit impatient tonight and want to cut a release based on this, before the week starts. I feel that while this is a big change in the code base (it's quite a big diff), it is not a controversial change and it doesn't change too much from the user's perspective.
The primary goal of this change is to extract the configuration logic from the import and export code. This adds a layer of abstraction and makes things simpler. Using this new power, some minor behaviour was improved. Issue #30 is partially (or fully?) solved.
I also took this opportunity to do some other clean up tasks. I added and fixed some comments. And I put some general utility code in a utility module.
I left detailed notes of exactly everything that changed in the commit logs.