launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #12394
Re: [Merge] lp:~adeuring/launchpad/sharingjob-remove-bp-subscriptions into lp:launchpad
On Mon, Sep 24, 2012 at 2:16 PM, Abel Deuring
<abel.deuring@xxxxxxxxxxxxx> wrote:
> Benji, thanks for these suggestions. I chose the first variant, moved the "if
> test_type == " blocks into the callsites.
> _assert_artifact_change_unsubscribes() is still quite long, but at least a
> bit more readable...
Cool.
--
https://code.launchpad.net/~adeuring/launchpad/sharingjob-remove-bp-subscriptions/+merge/126008
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
References