This is a bit weird because: - this is not testing anything, just setting a parameter, so maybe it should just be a helper method. - this uses a feature that is only present in trunk but this code runs on packages built from 1.2 as well… it won't blow up in 1.2 but it will set a config item that is not used anywhere. -- https://code.launchpad.net/~matsubara/maas/set-proxy/+merge/151074 Your team MAAS Maintainers is requested to review the proposed merge of lp:~matsubara/maas/set-proxy into lp:~maas-maintainers/maas/qa-lab-tests.
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
