Code review comment for lp:~mwhudson/launchpad/code-import-optimizations

Revision history for this message
Tim Penhey (thumper) wrote :

The change looks fine.

It does get me wondering a bit why we didn't seem to get any logging when it was pushing the branch to the central store before. Thanks for adding the job started and finished logging bits too.

I agree that it makes more sense to use branch rather than working tree in the workers.

review: Approve

« Back to merge proposal