This is definitely a step in the right direction. I don't have any issues with the code as such, but it still feels awkward. Some of the existing structure implies there is still execution of command line functions, which there aren't. Should we open an issue to clean this up or is it not really worth it?
Originally posted by @kkoz in #172 (review)
See also 5f12456.
This is definitely a step in the right direction. I don't have any issues with the code as such, but it still feels awkward. Some of the existing structure implies there is still execution of command line functions, which there aren't. Should we open an issue to clean this up or is it not really worth it?
Originally posted by @kkoz in #172 (review)
See also 5f12456.