RetroSearch Browse

Mon Mar 7 19:04:11 CET 2011 · http://mail.python.org/pipermail/python-dev/2011-March/108907.html

On Mar 07, 2011, at 06:31 PM, Antoine Pitrou wrote:

>On Mon, 7 Mar 2011 12:04:18 -0500
>Barry Warsaw <barry at python.org> wrote:
>> On Mar 07, 2011, at 11:44 AM, Tres Seaver wrote:
>> 
>> >If we can get to a mode where non-committers can push to a "fork" on
>> >hg.python.org, we can dodge the patch format issue by having folks post
>> >"pull requests" for that fork instaed.
>> >
>> >For the repoze and pylons projects, we have found the quality and
>> >quantity of patches went up *significantly* when we made it easy for
>> >somebody who doesn't work on the code all the time to use this workflow
>> >(fork to a public repo, clone, hack, commit, push, request a pull).
>> 
>> +1.  'Branches' are better than patches.
>
>How do you review a branch?

You can merge it locally and look at the diff.  Or use Rietveld if that's
supported.  But the reason a branch is better is because it's easier to track
the submitter's changes in response to your review comments, and it's easier
to make changes to their branch and push an update for *them* to see.

It's easier to have a ongoing conversation about a branch than a patch.

-Barry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/python-dev/attachments/20110307/5385cf65/attachment.pgp>

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from http://mail.python.org/pipermail/python-dev/2011-March/108907.html below:

[Python-Dev] hg diff