-
Notifications
You must be signed in to change notification settings - Fork 41
Open
Description
The Python files in PM-GUI need to be re-worked to be understood and edited more easily.
The current "module" (which is not one yet) is split in 3 parts (per @kloptops ):
- pugwash - most of the glue that binds the gui to the package manager
- pylibs/pugthemes.py - the craziest theme engine
- pylibs/pugscenes.py - the actual different scenes that are presented
Various improvements that I have from the top of my head:
- Make PM-GUI a real Python module, not using crazy relative imports
- Remove all usage of
__builtins__to set global variables. Prefer dependency injection and/or a shared settings object to global variables - Use
parseargsinstead of manually parsing string arguments - Standardize Python version and module requirements
- Setup minimal automated tests, at least testing that the UI launches
- make
pretend_devicea CLI argument - Factor code between
pugwashandharbourmaster
Metadata
Metadata
Assignees
Labels
No labels