The patch seems good, but for WorkingDir, the export function still uses 
attr.asdict ; so we cannot completely remove attr yet.

I guess replacing attr.asdict(self) by self.__dict__ would be an option but I 
remember mwhudson advising against the approach.

Maybe enumerating all elements is cleaner.
-- 
https://code.launchpad.net/~dbungert/curtin/+git/curtin/+merge/436274
Your team curtin developers is requested to review the proposed merge of 
~dbungert/curtin:no-attrs into curtin:master.


-- 
Mailing list: https://launchpad.net/~curtin-dev
Post to     : curtin-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~curtin-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to