fix: nil check for github.NewClient (#2465)

Fixes #2464

Author: chingor13

internal/github/github.go

@@ -59,7 +59,7 @@ func NewClient(accessToken string, repo *Repository) *Client {
 
 func newClientWithHTTP(accessToken string, repo *Repository, httpClient *http.Client) *Client {
 	client := github.NewClient(httpClient)
-	if repo.BaseURL != "" {
+	if repo != nil && repo.BaseURL != "" {
 		baseURL, _ := url.Parse(repo.BaseURL)
 		// Ensure the endpoint URL has a trailing slash.
 		if !strings.HasSuffix(baseURL.Path, "/") {

internal/github/github_test.go

@@ -1118,3 +1118,42 @@ func TestCreateTag(t *testing.T) {
 		})
 	}
 }
+
+func TestNewClient(t *testing.T) {
+
+	t.Parallel()
+	for _, test := range []struct {
+		name  string
+		token string
+		repo  *Repository
+	}{
+		{
+			name:  "with credentials",
+			token: "some-token",
+		},
+		{
+			name:  "with custom repo",
+			token: "some-token",
+			repo: &Repository{
+				Owner:   "some-owner",
+				Name:    "some-repo",
+				BaseURL: "https://example.com/",
+			},
+		},
+		{
+			name:  "with repo, without base url",
+			token: "some-token",
+			repo: &Repository{
+				Owner: "some-owner",
+				Name:  "some-repo",
+			},
+		},
+	} {
+		t.Run(test.name, func(t *testing.T) {
+			client := NewClient(test.token, nil)
+			if client == nil {
+				t.Fatalf("expected to create a new client")
+			}
+		})
+	}
+}