This doesn't add the remote either :)


On Thu, Aug 7, 2014 at 4:00 PM, Peter Muir <pmuir@redhat.com> wrote:

What Brian is doing is better IMO. With your approach you avoid a few commands but its just syntactic sugar. You still end up with the same thing in your local git (the remote added and the branch downloaded).

The PR branch is already in the upstream remote but not fetched by default. Brian is fetching from that, avoiding adding the remote.

Though nowadays I've been converted to the way of the big green button.

On 7 Aug 2014 19:08, "Lincoln Baxter, III" <lincolnbaxter@gmail.com> wrote:

Whatever works for you :) 

You don't need to add the remote, just copy the text from Github... it tells you how to do it right in the PR (just click "Use thh command line" to see the instructions)

git checkout -b lincolnthree-WINDUP-133 master
git pull https://github.com/lincolnthree/windup.git WINDUP-133

Then run the tests "mvn clean install", and if they pass:

git checkout master
git merge --no-ff lincolnthree-WINDUP-133
git push origin master

Hope this helps,
~Lincoln

On Sat, Aug 2, 2014 at 7:20 AM, Ondrej Zizka <ozizka@redhat.com> wrote:
Here is a script which Brian uses to rebase and merge selected PR's.
Could be handy.

HTH,
Ondra




-------- Original Message --------
Subject: Re: Git pull request automation?
Date: Tue, 30 Apr 2013 09:47:20 -0500
From: Brian Stansberry <brian.stansberry@redhat.com>
To: Ondrej Zizka <ozizka@redhat.com>
CC: Carlo de Wolf <cdewolf@redhat.com>


AIUI Carlo's team is building some automation around this for EAP.

I have a simple script I use to bring down a bunch of pull requests into 
a single branch.[1] I then push that branch to upstream master-ignore. 
Lightning and Tomaz Cerar's team city test that branch whenever changes 
are made. If that goes well, I merge the branch to master and push.

The other mergers do similar things, probably with better scripts for 
the first bit.

I don't use the requester's remote; the PR is available via the wildfly 
github repo. See the script for details.

[1]

pull() {
  cmd="git fetch upstream master "
  for var in "$@"
  do
    cmd="$cmd pull/$var/head:pullRequest$var"
  done

  $cmd

  git branch -D pulls
  git checkout master
  git rebase upstream/master
  git checkout -b pulls

  for var in "$@"
    do
    git checkout pullRequest$var
    git rebase pulls
    git checkout pulls
    git merge pullRequest$var
    git branch -D pullRequest$var
  done
}

On 4/30/13 9:16 AM, Ondrej Zizka wrote:
> Hi Brian,
>
> if I want to test and review the pull request before merging, it seems
> that I have to add the requester's remote, fetch it, check out his
> branch, then do what I want.
>
> With the pace of PRs to AS, I assume you have some automation for this,
> perhaps through GitHub API. Is that available somewhere?
> Or how do you proceed when you need to check something in an IDE, or
> test something not covered by the testsuite, etc?
>
> Thanks,
> Ondra


-- 
Brian Stansberry
Principal Software Engineer
JBoss by Red Hat



_______________________________________________
windup-dev mailing list
windup-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/windup-dev



--
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."

_______________________________________________
windup-dev mailing list
windup-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/windup-dev



--
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."