← Back to team overview

launchpad-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~rharding/launchpad/bp_default_1062207 into lp:launchpad

To: mp+129001@xxxxxxxxxxxxxxxxxx
From: Aaron Bentley <aaron@xxxxxxxxxxxxx>
Date: Wed, 10 Oct 2012 19:08:18 -0000
In-reply-to: <20121010174525.28950.31355.launchpad@ackee.canonical.com>
Reply-to: mp+129001@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

At first, I thought that this would always set the value to PUBLIC, but it only does so if the context is not a Product, or the Product's default is PUBLIC.  So this is a good change.
-- 
https://code.launchpad.net/~rharding/launchpad/bp_default_1062207/+merge/129001
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

References

[Merge] lp:~rharding/launchpad/bp_default_1062207 into lp:launchpad
From: Richard Harding, 2012-10-10