Code review comment for lp://staging/~unit193/bzr-fastimport/deletion-fixes

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Please add some unit tests for it.

It's the kind of change that's likely to regress as other changes are made, and it is not clear to me that it won't have other fallout (and e.g. make files disappear that should be exported) - tests are a good way to demonstrate that.

« Back to merge proposal