[PATCH] D67262: [git-llvm] Do not reinvent `@{upstream}`

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 12:37:19 PDT 2019


thakis added a comment.

I don't know what the long-term plans for this script are. I don't have a strong opinion, but weakly agree that it'd be good if it went away. In that case, I personally wouldn't add a warning and just make it temporarily do the (usually :P) right thing of using origin/master as upstream if there's no explicit upstream. If there's a warning however, it'd be nice if it included `git checkout -b myNewBranch origin/master` as suggested command to run, since I found that difficult to find. (Thanks for mentioning it!)

Alternatively, accepting something like `git llvm push origin` like mehdi suggested further up sounds good to me.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67262/new/

https://reviews.llvm.org/D67262





More information about the llvm-commits mailing list