Re: [Mutopia-discuss] update of Sicilienne (Op 78) by Gabriel Fauré

2012-12-09 Thread Glen Larsen
My last push did the same thing --- merged my push into my most recent pull
request. I'm familiar enough with git but need to figure out how to control
the github behavior.

-glen


On Sun, Dec 9, 2012 at 12:49 PM, Chris Sawer ch...@mutopiaproject.orgwrote:

 On 04/12/12 08:07, Federico Bruni wrote:

 I've just committed it to my repo and sent the pull request.
 Apparently, it has been added to the first pull request (still to be
 merged, AFAICS) as a comment and a link to the commit:

 https://github.com/chrissawer/**The-Mutopia-Project/pull/1https://github.com/chrissawer/The-Mutopia-Project/pull/1

 https://github.com/fedelibre/**The-Mutopia-Project/commit/**
 19054a818345793a744295c2d9886e**08bfbfa2c8https://github.com/fedelibre/The-Mutopia-Project/commit/19054a818345793a744295c2d9886e08bfbfa2c8


 Thanks, I have merged your pull request today and pushed the update to the
 Mutopia website.

 Not quite sure why the Sicilienne was added to the same pull request as
 Red Apple Rag. In the future separate pull requests for separate pieces (or
 at least separate composers) would make it easier for me to do updates in
 bitesize chunks.

 Regards,

 Chris

___
Mutopia-discuss mailing list
Mutopia-discuss@mutopiaproject.org
http://lists.bcn.mythic-beasts.com/mailman/listinfo/mutopia-discuss

Re: [Mutopia-discuss] update of Sicilienne (Op 78) by Gabriel Fauré

2012-12-09 Thread Chris Sawer
On 09/12/12 21:24, Felix Janda wrote:
 Would it be possible to send new/updated pieces as patches (git format-email)
 to the mailing list (to be imported with git am)? Then you can take one patch
 at the time.

Yes, although pull requests are more convenient and they also integrate 
nicely into the GitHub issue tracker.

If I keep up-to-date with pull requests then it won't be much of an 
issue.  :-)

Chris



___
Mutopia-discuss mailing list
Mutopia-discuss@mutopiaproject.org
http://lists.bcn.mythic-beasts.com/mailman/listinfo/mutopia-discuss


Re: [Mutopia-discuss] update of Sicilienne (Op 78) by Gabriel Fauré

2012-12-09 Thread Glen Larsen
Ah, thanks for the hint, Felix. if you have a pull request from your fork's
master any subsequent commit before that pull request is handled is
considered part of that pull request. I was creating branches on my local
repository, then merging them locally. I need to adjust my workflow.


On Sun, Dec 9, 2012 at 1:35 PM, Chris Sawer ch...@mutopiaproject.orgwrote:

 On 09/12/12 21:24, Felix Janda wrote:
  Would it be possible to send new/updated pieces as patches (git
 format-email)
  to the mailing list (to be imported with git am)? Then you can take one
 patch
  at the time.

 Yes, although pull requests are more convenient and they also integrate
 nicely into the GitHub issue tracker.

 If I keep up-to-date with pull requests then it won't be much of an
 issue.  :-)

 Chris



 ___
 Mutopia-discuss mailing list
 Mutopia-discuss@mutopiaproject.org
 http://lists.bcn.mythic-beasts.com/mailman/listinfo/mutopia-discuss

___
Mutopia-discuss mailing list
Mutopia-discuss@mutopiaproject.org
http://lists.bcn.mythic-beasts.com/mailman/listinfo/mutopia-discuss