Review: Needs Information Even though this branch is a little bit large, it looks like fantastic work, and I think that you've managed to reduce the complexity as well as the LOC count is impressive.
My only concern with this branch is that you have removed code from branchlookup which would only return public branches -- are you concerned about leaking information there? Since I am, and the new code does not seem to have anything like it, so could you ally my concerns? -- https://code.launchpad.net/~abentley/launchpad/all-url-support/+merge/113294 Your team Launchpad code reviewers is subscribed to branch lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
