From fdf2ce0b4dc90be32dd8bc3779513915421b9d65 Mon Sep 17 00:00:00 2001 From: Tom Maenan Read Cutting Date: Fri, 13 Oct 2017 16:10:44 +0100 Subject: [PATCH 1/3] Make `git open issue` default to upstream remote Fixes issue #95 --- git-open | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/git-open b/git-open index 6320f26..ac6bad0 100755 --- a/git-open +++ b/git-open @@ -23,8 +23,12 @@ if [[ "$1" == 'issue' ]]; then shift fi -# assume remote origin if not provided -remote=${1:-"origin"} +# assume remote origin if not provided (unless we are opening an issue, then use upstream) +if (( is_issue )); then + remote=${1:-"upstream"} +else + remote=${1:-"origin"} +fi # @TODO ls-remote will also expand "insteadOf" items `giturl=$(git ls-remote --get-url $remote)`` giturl=$(git config --get "remote.${remote}.url") From 239fb4df99e7226890b190aaef0a20d60c8f420a Mon Sep 17 00:00:00 2001 From: Tom Maenan Read Cutting Date: Mon, 16 Oct 2017 20:40:56 +0100 Subject: [PATCH 2/3] Make sure `git open issue` uses `upstream` only if exists --- git-open | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/git-open b/git-open index ac6bad0..9978261 100755 --- a/git-open +++ b/git-open @@ -26,6 +26,11 @@ fi # assume remote origin if not provided (unless we are opening an issue, then use upstream) if (( is_issue )); then remote=${1:-"upstream"} + giturl=$(git config --get "remote.${remote}.url") + # @TODO ls-remote will also expand "insteadOf" items `giturl=$(git ls-remote --get-url $remote)` + if [[ -z "$giturl" ]]; then + remote=${1:-"origin"} + fi else remote=${1:-"origin"} fi From f532cebb6b5bade7049490b7c227c5e15bb2bcea Mon Sep 17 00:00:00 2001 From: Tom Maenan Read Cutting Date: Mon, 16 Oct 2017 20:44:19 +0100 Subject: [PATCH 3/3] Fix indent size --- git-open | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/git-open b/git-open index 9978261..00364a8 100755 --- a/git-open +++ b/git-open @@ -29,7 +29,7 @@ if (( is_issue )); then giturl=$(git config --get "remote.${remote}.url") # @TODO ls-remote will also expand "insteadOf" items `giturl=$(git ls-remote --get-url $remote)` if [[ -z "$giturl" ]]; then - remote=${1:-"origin"} + remote=${1:-"origin"} fi else remote=${1:-"origin"}