Pull requests with patches

Jeremy Kerr jk at ozlabs.org
Fri Feb 11 14:37:32 EST 2011


Hi Guilherme,

> Oh, indeed. Haven't noticed that.  I always use two additional levels.
> However, I think, whitespace changes on lines that don't change
> otherwise are best placed in separate commits -- easier to review.

For changes like this, I'd prefer to keep it consistent with the existing 
formatting. We can revisit things later if it'd help though.

Also, as Dirk has suggested, would you be able to send your patches with 
something like git-format-patch and git-send-email? Having a proper changelog 
entry helps me enormously, as does including test cases where appropriate 
(don't worry about a testcase for this, but the pull request change could 
definitely use one).

Cheers,


Jeremy



More information about the Patchwork mailing list