fix(memlimit): don't use format string with slog (#19)

rhafer · web-flow · commit 0eced331ae17 · 2024-05-17T21:54:05.000+09:00
diff --git a/memlimit/memlimit.go b/memlimit/memlimit.go
@@ -167,7 +167,7 @@ func SetGoMemLimitWithOpts(opts ...Option) (_ int64, _err error) {
 	limit, err := setGoMemLimit(ApplyRatio(cfg.provider, ratio))
 	if err != nil {
 		if errors.Is(err, ErrNoLimit) {
-			cfg.logger.Info("memory is not limited, skipping: %v", err)
+			cfg.logger.Info("memory is not limited, skipping")
 			return 0, nil
 		}
 		return 0, fmt.Errorf("failed to set GOMEMLIMIT: %w", err)

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,7 @@ func SetGoMemLimitWithOpts(opts ...Option) (_ int64, _err error) {`
`167`	`167`	`limit, err := setGoMemLimit(ApplyRatio(cfg.provider, ratio))`
`168`	`168`	`if err != nil {`
`169`	`169`	`if errors.Is(err, ErrNoLimit) {`
`170`		`- cfg.logger.Info("memory is not limited, skipping: %v", err)`
	`170`	`+ cfg.logger.Info("memory is not limited, skipping")`
`171`	`171`	`return 0, nil`
`172`	`172`	`}`
`173`	`173`	`return 0, fmt.Errorf("failed to set GOMEMLIMIT: %w", err)`