all: fix nolint format

The valid syntax to suppress linter warnings is "//nolint:<name>" with
no space between comment and "nolint" and between ":".  Also, we move the
placement of nolint directive to the top of statements for multiple
nolint in the same scope.

While at it, fix and supress some linter warnings.

1 files changed, 4 insertions, 3 deletions

diff --git a/lib/git/git.go b/lib/git/git.go
index 1fcf2c97..7dd86d15 100644
--- a/lib/git/git.go
+++ b/lib/git/git.go
@@ -22,9 +22,10 @@ const (
 	_defBranch     = "master"
 )
 
+//nolint:gochecknoglobals
 var (
-	_stdout = os.Stdout //nolint: gochecknoglobals
-	_stderr = os.Stderr //nolint: gochecknoglobals
+	_stdout = os.Stdout
+	_stderr = os.Stderr
 )
 
 //
@@ -204,7 +205,7 @@ func GetRemoteURL(repoDir, remoteName string) (url string, err error) {
 		return
 	}
 
-	url, ok := gitIni.Get("remote", remoteName, "url")
+	url, ok := gitIni.Get("remote", remoteName, "url", "")
 	if !ok {
 		err = fmt.Errorf("GetRemote: Empty or invalid remote name")
 	}