A huge amount of the UDD importer's interesting code is in one file,
icommon.py.

I'd like to submit a series of changes to break it up such that only the
most common bits of code remain there.

Things I'm thinking of:

1) Move code in icommon.py that is uniquely invoked by a single
executable component of the importer out to that component:

e.g.
class ImportList, class PackageToImport --> import_package.py

class SubprocessMonitor, class ThreadDriver --> mass_import.py

load_explanations --> categorise_failures.py

2) Where a category of code with a single theme exists, move it from
icommon.py to a new file - e.g.:

move all code dealing with the Launchpad API to a new ilpapi.py.

move all code dealing with sqlite dbs to a new idatabase.py.



Assuming the idea is liked, I think the best way to proceed would be for
me to do a single one of these moves, merge-propose it, wait for
approval, land it, and repeat the cycle until done.

This should minimize clashing with other branches trying to land, and
potentially awkward merges.


Max.




Attachment: signature.asc
Description: OpenPGP digital signature

-- 
ubuntu-distributed-devel mailing list
ubuntu-distributed-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-distributed-devel

Reply via email to