launchpad-reviewers team mailing list archive

[Abel Deuring <abel.deuring@xxxxxxxxxxxxx>]

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...
-- 
https://code.launchpad.net/~adeuring/launchpad/sharingjob-remove-bp-subscriptions/+merge/126008
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.