Merge pull request #80 from nkmr-jp/develop

Upgrade dependencies

.github/workflows/lint.yml

@@ -17,9 +17,9 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: '1.22.x'
+          go-version: '1.23.x'
           cache: false
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v6
         with:
-          version: v1.54.1
+          version: v1.62.2

.github/workflows/test.yml

@@ -14,9 +14,9 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go: ["1.21.x", "1.22.x"]
+        go: ["1.22.x", "1.23.x"]
         include:
-          - go: 1.22.x
+          - go: 1.23.x
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -27,4 +27,4 @@ jobs:
       - name: Test
         run: make cover
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v4
+        uses: codecov/codecov-action@v5

.go-version

@@ -1 +1 @@
-1.21.0
+1.23.2

go.mod

@@ -6,7 +6,7 @@ require (
 	github.com/davecgh/go-spew v1.1.1
 	github.com/logrusorgru/aurora/v4 v4.0.0
 	github.com/samber/lo v1.47.0
-	github.com/stretchr/testify v1.9.0
+	github.com/stretchr/testify v1.10.0
 	go.uber.org/zap v1.27.0
 	gopkg.in/natefinch/lumberjack.v2 v2.2.1
 )

go.sum

@@ -6,8 +6,8 @@ github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZb
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
 github.com/samber/lo v1.47.0 h1:z7RynLwP5nbyRscyvcD043DWYoOcYRv3mV8lBeqOCLc=
 github.com/samber/lo v1.47.0/go.mod h1:RmDH9Ct32Qy3gduHQuKJ3gW1fMHAnE/fAzQuf6He5cU=
-github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg=
-github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
+github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
+github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
 go.uber.org/goleak v1.3.0 h1:2K3zAYmnTNqV73imy9J1T3WC+gmCePx2hEGkimedGto=
 go.uber.org/goleak v1.3.0/go.mod h1:CoHD4mav9JJNrW/WLlf7HGZPjdw8EucARQHekz1X6bE=
 go.uber.org/multierr v1.11.0 h1:blXXJkSxSSfBVBlC76pxqeO+LN3aDfLQo+309xJstO0=

pretty.go

@@ -60,7 +60,7 @@ func (l *prettyLogger) logWithError(msg string, level zapcore.Level, err error,
 	err2 := l.Logger.Output(
 		4,
 		l.coloredLevel(level).String()+" "+l.coloredMsg(
-			fmt.Sprintf("%s%s%s", msg, separator, au.Magenta(err.Error())),
+			fmt.Sprintf("%s%s%s", msg, separator, au.Magenta(fmt.Sprintf("%v", err))),
 			level, fields,
 		),
 	)

pretty_test.go

@@ -235,6 +235,16 @@ func Test_prettyLogger_logWithError(t *testing.T) {
 		l.logWithError("test message", zapcore.InfoLevel, errors.New("some error"), nil)
 		assert.Contains(t, buf.String(), "[INTERNAL ERROR] ")
 	})
+
+	t.Run("when error is nil", func(t *testing.T) {
+		outputType = PrettyOutput
+		severityLevel = zapcore.DebugLevel
+
+		var buf bytes.Buffer
+		l := newPrettyLogger(&buf, os.Stderr)
+		l.logWithError("test message", zapcore.InfoLevel, nil, nil)
+		assert.Contains(t, buf.String(), "<nil>")
+	})
 }
 
 func Test_prettyLogger_coloredLevel(t *testing.T) {