On Thu, 9 Mar 2017 12:01:08 -0600, Matt Sicker wrote:
Try viewing the log like this:

git log --oneline --decorate --graph

It might be farther back in time,

Oh yes, of course, you are right.
I tested this with a fairly old pull request!

Thanks and sorry for the noise,
Gilles

so the added commits won't show up at the
top of the normal git log format.

On 9 March 2017 at 11:48, Gilles <gil...@harfang.homelinux.org> wrote:

On Thu, 9 Mar 2017 11:34:42 -0600, Matt Sicker wrote:

A merge commit is an additional commit unless you squash what you're merging. The entire history should be available. You can use --ff to only allow a fast forward commit so it doesn't add a merge commit, but then the branch you're merging has be to rebased on the one you're merging into.


I did

$ git pull github pull/50/head

in a local copy of Commons Math "master" branch.

The command performed a single commit (a "merge") and there is no
trace that the change(s) were coming from someone else (a GitHub user):

  https://github.com/apache/commons-math/pull/50/commits/b32cd
65abb0497fa35ca407430f20d562f604ed7

$ git log
commit 68fdf0620ecfc708a940cfd02451986cc42fde5b
Merge: 37c3d36c8 b32cd65ab
Author: Gilles <er...@apache.org>
Date:   Thu Mar 9 16:01:07 2017 +0100

Merge commit 'refs/pull/50/head' of https://github.com/apache/comm
ons-math into fix-MATH-1402

[No other commit related to that action.]


Gilles




On 9 March 2017 at 09:08, Gilles <gil...@harfang.homelinux.org> wrote:

On Thu, 9 Mar 2017 09:38:55 -0500, Rob Tompkins wrote:

On Mar 9, 2017, at 8:08 AM, Gilles <gil...@harfang.homelinux.org> wrote:


On Wed, 8 Mar 2017 10:37:44 -0600, Matt Sicker wrote:

The "write to a PR branch" was something I mentioned in a different
email.

If you want a single command to merge a PR into master, assuming
you've
already set up "github" as your github remote:


$ git remote -v
github    https://github.com/apache/commons-math.git (fetch)
github    https://github.com/apache/commons-math.git (push)
origin https://git-wip-us.apache.org/repos/asf/commons-math.git
(fetch)
origin https://git-wip-us.apache.org/repos/asf/commons-math.git
(push)


git pull github pull/42
git push

Something like that should work.


$ git pull github pull/50
fatal: Couldn't find remote ref pull/50

Am I missing something?


Try

git pull github pull/#/head


It successfully performs a merge into the current branch, but does
not preserve the commit history (only the merge appears in "git log").

IIRC, the generated instructions[1] would lead to the contributor's
commit being referenced properly.

Regards,
Gilles

[1] Such as in:
      https://issues.apache.org/jira/browse/NUMBERS-4




-Rob

Regards,

Gilles

[...]







---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to