launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:close-account-import-policy into launchpad:master.

Commit message:
Add close_account to lp.registry.scripts.closeaccount.__all__

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/401125

Fixes:

    There were 1 imports of names not appearing in the __all__.
    You should not import close_account from lp.registry.scripts.closeaccount:
        lp.registry.model.persontransferjob
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:close-account-import-policy into launchpad:master.

diff --git a/lib/lp/registry/scripts/closeaccount.py b/lib/lp/registry/scripts/closeaccount.py
index 6fa8585..27b2eb1 100644
--- a/lib/lp/registry/scripts/closeaccount.py
+++ b/lib/lp/registry/scripts/closeaccount.py
@@ -4,7 +4,10 @@
 """Remove personal details of a user from the database, leaving a stub."""
 
 __metaclass__ = type
-__all__ = ['CloseAccountScript']
+__all__ = [
+    'close_account',
+    'CloseAccountScript',
+    ]
 
 import six
 from storm.exceptions import IntegrityError