[Merge] ~ddstreet/software-properties:lp645404 into software-properties:ubuntu/master
Dan Streetman
ddstreet at canonical.com
Sat Jan 25 14:02:07 UTC 2020
> I don't like this commit, it makes it harder to read for basically no benefit:
> move 'datadir' into kwargs, set default in SoftwareProperties
>
> same for rootdir moving into kwargs in the next commit. kwargs should be
> avoided as much as humanely possible. It prevents a lot of static checks
> (especially type checks get harder, and I'd love to add some mypy annotations
> at some point).
>
> The removal of the Options stuff I don't like very much, I felt it was cleaner
> before.
>
>
> Refactoring of ShortCutHandler I can't really follow, it's too big.
Here's a different approach that makes the absolute minimum changes to SoftwareProperties, and splits up much of the rewriting to possibly be easier for you to review.
Just an initial draft, I need to go over the commits again and check the test updates, so it's not ready to merge yet but shoudl be ready for general review.
--
https://code.launchpad.net/~ddstreet/software-properties/+git/software-properties/+merge/367276
Your team Ubuntu Core Development Team is requested to review the proposed merge of ~ddstreet/software-properties:lp645404 into software-properties:ubuntu/master.
More information about the Ubuntu-reviews
mailing list