Code review comment for lp:~allenap/launchpad/do-copy-async-bug-766247

Revision history for this message
Brad Crittenden (bac) wrote :

Hi Gavin,

This change makes sense and the tests look good.

Two little things:

* type "copyingto"

* If the XXX is something we really need to do why not file a bug so we don't forget?

review: Approve (code)

« Back to merge proposal