From c562bb4de6159eeef10c2bb52bb76b7a1934d60b Mon Sep 17 00:00:00 2001 From: cmccandless Date: Mon, 25 Sep 2017 10:57:21 -0500 Subject: [PATCH 1/2] Define default remote for repository at "gerrit.default-remote" --- lib/cli.js | 4 ++++ lib/gerrit.js | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/cli.js b/lib/cli.js index 37bb0c6..e7d4086 100644 --- a/lib/cli.js +++ b/lib/cli.js @@ -41,6 +41,10 @@ cli.config = function(name, options) { } else { + + if (!git.config("gerrit.default-remote")) { + git.config("gerrit.default-remote", "origin"); + } name = name || "default"; diff --git a/lib/gerrit.js b/lib/gerrit.js index 61d3bfa..0826923 100644 --- a/lib/gerrit.js +++ b/lib/gerrit.js @@ -22,7 +22,7 @@ gerrit.GerritError = GerritError; gerrit.parseRemote = function(remote) { - remote = remote || "origin"; + remote = remote || git.config("gerrit.default-remote") || "origin"; return requireInRepo() .then(function() { From 178cd7da448e4bd5d23e86f57eef210b07f46e95 Mon Sep 17 00:00:00 2001 From: Corey McCandless Date: Fri, 13 Oct 2017 10:20:28 -0500 Subject: [PATCH 2/2] specify local scope for gerrit-default-remote --- lib/cli.js | 2 +- lib/gerrit.js | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/lib/cli.js b/lib/cli.js index e7d4086..87af095 100644 --- a/lib/cli.js +++ b/lib/cli.js @@ -43,7 +43,7 @@ cli.config = function(name, options) { else { if (!git.config("gerrit.default-remote")) { - git.config("gerrit.default-remote", "origin"); + git.config("gerrit.default-remote", "origin", {local: true}); } name = name || "default"; diff --git a/lib/gerrit.js b/lib/gerrit.js index 0826923..11d8c9b 100644 --- a/lib/gerrit.js +++ b/lib/gerrit.js @@ -22,7 +22,12 @@ gerrit.GerritError = GerritError; gerrit.parseRemote = function(remote) { - remote = remote || git.config("gerrit.default-remote") || "origin"; + var defaultRemote = git.config("gerrit.default-remote", {local: true}); + if (!defaultRemote) { + defaultRemote = "origin"; + git.config("gerrit.default-remote", defaultRemote, {local: true}); + } + remote = remote || defaultRemote || "origin"; return requireInRepo() .then(function() {