Code review comment for lp:~ian-clatworthy/bzr/faster-log
Revision history for this message
| Ian Clatworthy (ian-clatworthy) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
This is an old patch that I still think we ought to land. The previous reviews asked for more tests because there's a small risk I might have broken something. I agree that additional tests would be nice. I disagree that they ought to block the patch because, IMO, the patch's value outweighs that risk.
Perhaps this weeks patch pilot could add some more tests if they still feel they are necessary?