[PATCH] Tests for KnitContent class (Revisited)
John Arbash Meinel
john at arbash-meinel.com
Tue Nov 28 15:10:50 GMT 2006
Dmitry Vasiliev wrote:
...
> I fully agree with the comments and in the new (attached) version of the
> patch I've decided to not change the signatures for now since
> KnitContent.line_delta(_iter) methods should be reviewed anyway (see my
> comments above).
>
>
Thanks for this. +1 from me, so I'm submitting it now.
Is there a reason you are submitting patches rather than bundles?
In general, we prefer bundles because:
1) Bundle Buggy can automatically track them. Including when they are
merged into bzr.dev, and when they are superseded by a follow up patch.
2) They allow us to give proper annotation to you for doing the changes.
3) It allows us to do 'bzr merge ../foo.patch' rather than 'patch -p0 <
../foo.patch'
This is a minor thing, but it does mean we can merge valid patches even
if the source has moved around a bit. (1) is probably the biggest
advantage at the moment. :)
John
=:->
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061128/f61fd181/attachment.pgp
More information about the bazaar
mailing list