[Launchpad-reviewers] [Merge] lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad

2016-10-13 Thread noreply
The proposal to merge lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~cjwatson/launchpad/codeimport-git-worker/+merge/308142 -- Your team Launchpad code reviewers is subscribed

Re: [Launchpad-reviewers] [Merge] lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad

2016-10-12 Thread William Grant
Review: Approve code Diff comments: > > === modified file 'lib/lp/codehosting/codeimport/worker.py' > --- lib/lp/codehosting/codeimport/worker.py 2016-10-11 15:36:32 + > +++ lib/lp/codehosting/codeimport/worker.py 2016-10-11 15:36:32 + > @@ -918,3 +973,67 @@ > """See `Pull

Re: [Launchpad-reviewers] [Merge] lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad

2016-10-12 Thread Colin Watson
I've sprinkled a bunch of extra comments around in various places. Diff comments: > > === modified file 'lib/lp/codehosting/codeimport/worker.py' > --- lib/lp/codehosting/codeimport/worker.py 2016-10-11 15:36:32 + > +++ lib/lp/codehosting/codeimport/worker.py 2016-10-11 15:36:32 + >

Re: [Launchpad-reviewers] [Merge] lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad

2016-10-12 Thread William Grant
Review: Approve code Diff comments: > === modified file 'configs/development/launchpad-lazr.conf' > --- configs/development/launchpad-lazr.conf 2016-06-30 16:05:11 + > +++ configs/development/launchpad-lazr.conf 2016-10-11 15:36:32 + > @@ -55,6 +55,7 @@ > > [codeimport] > bazaar

[Launchpad-reviewers] [Merge] lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad

2016-10-11 Thread Colin Watson
Colin Watson has proposed merging lp:~cjwatson/launchpad/codeimport-git-worker into lp:launchpad with lp:~cjwatson/launchpad/codeimport-create-hosting as a prerequisite. Commit message: Add a Git-to-Git import worker. Requested reviews: Launchpad code reviewers (launchpad-reviewers) Related b