fixing lint errors

nucleuscloud · Nov 8, 2023 · 033154b · 033154b
1 parent 13746c6
commit 033154b
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 18 deletions.
diff --git a/cli/internal/auth/tokens.go b/cli/internal/auth/tokens.go
@@ -3,6 +3,7 @@ package auth
 import (
 	"context"
 	"fmt"
+	"log/slog"
 	"net/http"
 	"net/url"
 	"time"
@@ -51,7 +52,7 @@ func getToken(ctx context.Context) (string, error) {
 		if err != nil {
 			return "", err
 		}
-		fmt.Println("access token is no longer valid. attempting to refresh...")
+		slog.Info("access token is no longer valid. attempting to refresh...")
 		refreshtoken, err := userconfig.GetRefreshToken()
 		if err != nil {
 			return "", err

diff --git a/cli/internal/cmds/neosync/login/login.go b/cli/internal/cmds/neosync/login/login.go
@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"log/slog"
 	"net/http"
 	"net/url"
 	"time"
@@ -40,7 +41,7 @@ func login(ctx context.Context) error {
 		return err
 	}
 	if !authEnabledResp.Msg.IsEnabled {
-		fmt.Println("auth is not enabled server-side, exiting")
+		slog.Info("auth is not enabled server-side, exiting")
 		return nil
 	}
 	return oAuthLogin(ctx, authclient)
@@ -93,7 +94,7 @@ func oAuthLogin(
 	}()
 
 	if err := webbrowser.Open(authorizeurlResp.Msg.Url); err != nil {
-		fmt.Println("There was an issue opening the web browser, proceed to the following url to finish logging in to Neosync:\n", authorizeurlResp.Msg.Url)
+		fmt.Println("There was an issue opening the web browser, proceed to the following url to finish logging in to Neosync:\n", authorizeurlResp.Msg.Url) //nolint
 	}
 
 	select {
@@ -119,7 +120,6 @@ func oAuthLogin(
 		if err != nil {
 			return err
 		}
-		fmt.Println("write file success!")
 		if loginResp.Msg.AccessToken.RefreshToken != nil {
 			err = userconfig.SetRefreshToken(*loginResp.Msg.AccessToken.RefreshToken)
 			if err != nil {

diff --git a/cli/internal/cmds/neosync/version/version.go b/cli/internal/cmds/neosync/version/version.go
@@ -29,20 +29,20 @@ func NewCmd() *cobra.Command {
 				if err != nil {
 					return err
 				}
-				fmt.Println(string(marshaled))
+				fmt.Println(string(marshaled)) //nolint
 			} else if output == "yaml" {
 				marshaled, err := yaml.Marshal(&versionInfo)
 				if err != nil {
 					return err
 				}
-				fmt.Println(string(marshaled))
+				fmt.Println(string(marshaled)) //nolint
 			} else {
-				fmt.Println("Git Version:", versionInfo.GitVersion)
-				fmt.Println("Git Commit:", versionInfo.GitCommit)
-				fmt.Println("Build Date:", versionInfo.BuildDate)
-				fmt.Println("Go Version:", versionInfo.GoVersion)
-				fmt.Println("Compiler:", versionInfo.Compiler)
-				fmt.Println("Platform:", versionInfo.Platform)
+				fmt.Println("Git Version:", versionInfo.GitVersion) //nolint
+				fmt.Println("Git Commit:", versionInfo.GitCommit)   //nolint
+				fmt.Println("Build Date:", versionInfo.BuildDate)   //nolint
+				fmt.Println("Go Version:", versionInfo.GoVersion)   //nolint
+				fmt.Println("Compiler:", versionInfo.Compiler)      //nolint
+				fmt.Println("Platform:", versionInfo.Platform)      //nolint
 			}
 			return nil
 		},

diff --git a/cli/internal/cmds/neosync/whoami/whoami.go b/cli/internal/cmds/neosync/whoami/whoami.go
@@ -42,6 +42,6 @@ func whoami(ctx context.Context) error {
 		return err
 	}
 	// todo: layer in account data and access/id token information for even more goodness
-	fmt.Println("UserId:", resp.Msg.UserId)
+	fmt.Println("UserId:", resp.Msg.UserId) // nolint
 	return nil
 }
diff --git a/cli/internal/connect/interceptors/auth/interceptor.go b/cli/internal/connect/interceptors/auth/interceptor.go
@@ -3,7 +3,7 @@ package auth_interceptor
 import (
 	"context"
 	"errors"
-	"fmt"
+	"log/slog"
 
 	"connectrpc.com/connect"
 )
@@ -44,7 +44,7 @@ func (i *Interceptor) WrapStreamingClient(next connect.StreamingClientFunc) conn
 			if err != nil {
 				err2 := conn.CloseRequest()
 				if err2 != nil {
-					fmt.Println(err2) // todo
+					slog.Info(err2.Error()) // todo
 				}
 				return conn
 			}

diff --git a/worker/internal/benthos/transformers/phone_test.go b/worker/internal/benthos/transformers/phone_test.go
@@ -1,7 +1,6 @@
 package neosync_transformers
 
 import (
-	"fmt"
 	"strings"
 	"testing"
 
@@ -60,8 +59,6 @@ func TestGeneratePhoneNumberPreserveLengthFalseIncludeHyphensTrue(t *testing.T)
 
 	res, err := GenerateRandomPhoneNumberWithHyphens()
 
-	fmt.Println("res", res)
-
 	assert.NoError(t, err)
 	assert.Equal(t, len(res), expectedLength, "The length of the output phone number should be the same as the input phone number")