Code review comment for lp://staging/~unit193/bzr-fastimport/deletion-fixes
Revision history for this message
| Richard Wilbur (richard-wilbur) wrote | # |
review:
Needs Information
(needs test(s))
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
2eb648a
demo site
(Get the code!)
Thanks Unit 193 for the patch. It looks well executed.
How hard do you think it would be to create a test (or tests) to demonstrate the problem(s) (absent the fix) and validate the fix? This is important to make sure no one else accidentally breaks this fix, reintroducing the problem(s), in the future.