Robert Collins wrote:
> On Fri, 2009-11-20 at 14:21 +0000, Aaron Bentley wrote:
>> This could easily be a cause of unwanted email, if every time the branch
>> is pushed to, new interdiffs are emailed.
>
> John suggests having a button to trigger this.
>
> I think that the resubmit button could be repurposed, as at the moment
> losing the conversation history is a misfeature for most folk I know.
We plan to show the conversation from previous merge proposals. I don't
think changing the meaning of resubmit would work.
> I will note though, that you don't have to push on every commit :).
>
> Oh, and the moving diff should be the diff between the previews
You mean a diff of the past and present preview diffs?
, not the
> changes along the branch - so merges from trunk and other 'things are
> not changed' pushes /will not/ trigger mail anyway, as the preview won't
> change.
I don't follow. Diffs include line numbers and context-- these may both
change when the target branch is merged into the source branch.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Robert Collins wrote:
> On Fri, 2009-11-20 at 14:21 +0000, Aaron Bentley wrote:
>> This could easily be a cause of unwanted email, if every time the branch
>> is pushed to, new interdiffs are emailed.
>
> John suggests having a button to trigger this.
>
> I think that the resubmit button could be repurposed, as at the moment
> losing the conversation history is a misfeature for most folk I know.
We plan to show the conversation from previous merge proposals. I don't
think changing the meaning of resubmit would work.
> I will note though, that you don't have to push on every commit :).
>
> Oh, and the moving diff should be the diff between the previews
You mean a diff of the past and present preview diffs?
, not the
> changes along the branch - so merges from trunk and other 'things are
> not changed' pushes /will not/ trigger mail anyway, as the preview won't
> change.
I don't follow. Diffs include line numbers and context-- these may both
change when the target branch is merged into the source branch.
Aaron enigmail. mozdev. org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAks G+j0ACgkQ0F+ nu1YWqI2JUACfUD ewkv77lZTUrM3mO 6ZQivVn o1InFjKhdD7nJlo b2
U04An2oDBHqMNzq
=fy+f
-----END PGP SIGNATURE-----