automactc icon indicating copy to clipboard operation
automactc copied to clipboard

updates to support installing automactc script via pip

Open ryandeivert opened this issue 5 years ago • 4 comments

to @kshitijkumar14

We are interested in leveraging some of the functionality of automactc, but cannot do so in its current form. Ideally, this would be installable via pip to support on-demand installation (when applicable). These changes are largely to support that.

I'm happy to answer any questions about the change or update as you see fit. With these changes, you will be able to publish to pypi.

Changes

  • Implementing modules as classes to remove executing of modules at import time.
    • This resulted in a significant amount of line changes, but the majority of the code itself has been unchanged.
  • Introducing ModuleRegistry for storing and accessing module classes.
  • Refactoring the CLI to reduce the use of globals and removing usage of __main__ imports from modules.
    • The API for the CLI remains unchanged - all arguments are the same, and the code is largely the same. I've introduced a CLIRunner class to help organize things and store variables.
  • Migrating python package files to within a top-level automactc folder to better support packaging.
  • Adding setuptools support (setup.py) to allow for installing via pip. This will result in a automactc executable being installed.
    • I have added placeholder info to the setup.py for now - this can be updated as desired prior to publishing to pypi.

ryandeivert avatar Apr 27 '19 08:04 ryandeivert

@kshitijkumar14 any chance I could get some feedback on this?

ryandeivert avatar May 02 '19 20:05 ryandeivert

Hi @ryandeivert - thanks a ton for these updates. I'll be reviewing these and will follow up with you.

kshitijkumar14 avatar May 02 '19 21:05 kshitijkumar14

thanks @kshitijkumar14 :) friendly ping again to keep this alive. any updates?

ryandeivert avatar May 14 '19 20:05 ryandeivert

hello @kshitijkumar14 .. just curious if this is still on your radar. I think a lot of people would benefit from this

ryandeivert avatar Aug 13 '19 21:08 ryandeivert