Username and email must be set

The values aren't super important, as we're only using them to support
merging branches in git and never pushing code.

pkg/vcs/client.go

@@ -9,6 +9,11 @@ import (
 	"github.com/zapier/kubechecks/pkg/repo"
 )
 
+const (
+	DefaultVcsUsername = "kubechecks"
+	DefaultVcsEmail    = "[email protected]"
+)
+
 var (
 	// ErrInvalidType is a sentinel error for use in client implementations
 	ErrInvalidType  = errors.New("invalid event type")

pkg/vcs/github_client/client.go

@@ -69,12 +69,26 @@ func CreateGithubClient() (*Client, error) {
 		return nil, errors.Wrap(err, "failed to get user")
 	}
 
-	return &Client{
+	client := &Client{
 		Client:   googleClient,
 		v4Client: shurcoolClient,
-		username: *user.Login,
-		email:    *user.Email,
-	}, nil
+	}
+	if user != nil {
+		if user.Login != nil {
+			client.username = *user.Login
+		}
+		if user.Email != nil {
+			client.email = *user.Email
+		}
+	}
+
+	if client.username == "" {
+		client.username = vcs.DefaultVcsUsername
+	}
+	if client.email == "" {
+		client.email = vcs.DefaultVcsEmail
+	}
+	return client, nil
 }
 
 func (c *Client) Username() string { return c.username }

pkg/vcs/gitlab_client/client.go

@@ -55,7 +55,14 @@ func CreateGitlabClient() (*Client, error) {
 		return nil, errors.Wrap(err, "failed to get current user")
 	}
 
-	return &Client{Client: c, username: user.Username, email: user.Email}, nil
+	client := &Client{Client: c, username: user.Username, email: user.Email}
+	if client.username == "" {
+		client.username = vcs.DefaultVcsUsername
+	}
+	if client.email == "" {
+		client.email = vcs.DefaultVcsEmail
+	}
+	return client, nil
 }
 
 func (c *Client) Email() string    { return c.email }