D grammar overhaul

Brad Roberts braddr at puremagic.com
Thu Mar 31 11:04:59 PDT 2011


On 3/31/2011 10:34 AM, Jonathan M Davis wrote:
> On 2011-03-31 02:32, Trass3r wrote:
>> Rainer Schuetze Wrote:
>>> How can I contribute patches? I guess this would be with pull
>>> requests, so I'll need a github repository, too? The discussion page for
>>> pull requests looks like a nice feature...
>>
>> Yep, you fork the original repo and apply your changes. Once you think
>> you're somewhat done you can open a pull request. I think you can even
>> comment on single lines in commits.
>>
>> The advantage of this approach is that your changes don't appear as a
>> single huge diff but as atomic changesets (as long as the author groups
>> changes that belong together in single commits)
> 
> I would point out though that pull requests are done by branch, so if you have 
> commits that you don't want to be grouped with others or put in the pull 
> request, then they need to be on a separate branch. If you go and make several 
> commits and then only want some of them pulled, that doesn't work so well. 
> It's technically feasible via git, but it's more of a pain, and it's not how 
> github itself manages the process. So, typically, you'd make a set of changes 
> on a branch created for those changes - be they one commit or several. Then 
> you do a pull request on that branch. Unrelated changes would be on other 
> branches.
> 
> - Jonathan M Davis

Um.. no.  Github allows you to pick both a subrange of a branch.  Take a look at the help docs:

   http://help.github.com/pull-requests/



More information about the Digitalmars-d mailing list