The State of Go

Re:The State of Go by justinmk on Saturday February 14, 2015 @07:01PM (#9051220)

> Don't use the merge button, just add the requester's repo as a remote to yours

Actually, you don't need to do that. GitHub provides a special pulls remote "namespace" on the upstream repo, so you can add it as a fetch pattern to your .git/config like so:

    [remote "upstream"]
      url = https://github.com/neovim/neovim.git
      fetch = +refs/heads/*:refs/remotes/upstream/*
      fetch = +refs/pull/*/head:refs/pull/upstream/*

Then when you `git fetch --all`, you will have ALL pull requests available in your local repo in the local pull/ namespace. To check out PR #42:

    git checkout -b foo refs/pull/upstream/42

[ Parent | on HN ]

Re:The State of Go by ChuckMcM on Saturday February 14, 2015 @07:49PM (#9051365)

Wow you are my new favorite person. What I was going to say was that the GP was correct in that it just took a bit more work on the maintainers part to keep the history clean, but even that is unnecessary apparently :-)

[ Parent | on HN ]

Re:The State of Go by eiopa on Saturday February 14, 2015 @09:47PM (#9051616)

Good tip.

It's interesting that other systems has a nicer implementation for it.

In Phabricator, it's:

  $ arc patch D12345

and it checks out revision 12345 into an appropriately named branch.

[ Parent | on HN ]

Re:The State of Go by jbish on Saturday February 14, 2015 @07:11PM (#9051260)

Yes! I've been wanting to do this and didn't know it was this easy.

[ Parent | on HN ]

Re:The State of Go by Mithaldu on Saturday February 14, 2015 @07:03PM (#9051225)

That is great, and i thank you for bringing that to my, and everyone else's attention! :)

[ Parent | on HN ]

Re:The State of Go by zak_mc_kracken on Saturday February 14, 2015 @10:50PM (#9051722)

This is... amazing. Thank you so much.

[ Parent | on HN ]

Re:The State of Go by config_yml on Sunday February 15, 2015 @09:29AM (#9052671)

is this documented somewhere on github?

[ Parent | on HN ]

Re:The State of Go by ben0x539 on Sunday February 15, 2015 @03:46PM (#9054038)

Yeah, I have to google the precise path every time I need it: https://help.github.com/articles/checking-out-pull-requests-...

[ Parent | on HN ]

Re:The State of Go by hhaidar on Monday February 16, 2015 @10:46AM (#9057367)

WOW, this has been missing from my life for way too long!

Thank you :D

[ Parent | on HN ]

Re:The State of Go by hacknat on Saturday February 14, 2015 @09:02PM (#9051538)

You rock!

[ Parent | on HN ]