launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #13949
Re: [Merge] lp:~wallyworld/launchpad/person-index-timeout-931771 into lp:launchpad
Review: Approve
This looks good, Ian, just one minor quibble you can deal with before landing.
#248: If this is meant to get around circular imports, please comment it as
such. It's also cleaner to move imports to the top of the execution scope
they're brought into.
--
https://code.launchpad.net/~wallyworld/launchpad/person-index-timeout-931771/+merge/132446
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
References