launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #16233
Re: [Merge] lp:~wgrant/launchpad/antibfjo-7-no-set-pls into lp:launchpad
Review: Approve code
170 + queues = [removeSecurityProxy(bq) for bq in queues]
You're overriding the same variable, I'd prefer one of them change.
734 === removed file 'lib/lp/soyuz/tests/test_buildpackagejob.py'
Good riddance.
--
https://code.launchpad.net/~wgrant/launchpad/antibfjo-7-no-set-pls/+merge/195894
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
References