News for hackers. Stuff that matters.
Re:Hub Git Wrapper by suprjami on Thursday May 28, 2015 @06:38AM (#9617136)
I once tried to PR from hub.

It sent the PR from my branch on my fork to master on my fork. Obviously I wished to PR against the original thing I forked from.

I just stick to the web interface now.

Re:Hub Git Wrapper by emacsfodder on Thursday May 28, 2015 @06:59AM (#9617206)
What you needed was to use the options -h (head, ie. the feature branch) and -b (base, ie the upstream master) eg.

    git pull-request \
      -m "I added an awesome feature"\
      -b baserepo:master\
      -h myfork:feature-branch
Re:Hub Git Wrapper by justinsb on Thursday May 28, 2015 @07:35AM (#9617347)
The GitHub web interface defaults to the PR-to-upstream behavior though, so this is at least an inconsistency; I would argue it is a bug.
Re:Hub Git Wrapper by ultrafez on Thursday May 28, 2015 @06:50AM (#9617177)
You could submit a pull request to hub to fix it!

Perhaps not using hub to do so, though.

Re:Hub Git Wrapper by emacsfodder on Thursday May 28, 2015 @07:00AM (#9617210)
hub works just fine to create a PR, some lack of RTFM is at play.
Re:Hub Git Wrapper by skj on Thursday May 28, 2015 @07:36AM (#9617353)
So, I never trust users to give me advice on features to add.

But I always believe them when they say something is difficult or confusing. Research should not be required to make a good CLI do what you want, provided you understand what you want.

Re:Hub Git Wrapper by Manishearth on Thursday May 28, 2015 @08:12AM (#9617533)
Re:Hub Git Wrapper by michaelmior on Thursday May 28, 2015 @07:55AM (#9617445)
I believe will always default to sending the PR to the remote named origin. If you have the original repo listed as origin and your repo as a remote with your username, you get the expected behaviour. I think the issue here is that hub has no explicit knowledge of the ancestry of the repo so unless it fetches this from the GH API, I think it's doing the only logical thing.
Re:Hub Git Wrapper by mgold on Thursday May 28, 2015 @10:19AM (#9618294)
It sure would be nice to have `--dry-run --verbose` to see exactly what it's going to do.