Copilot commented on code in PR #239:
URL: https://github.com/apache/skywalking-go/pull/239#discussion_r2786172034
##########
log/logger.go:
##########
@@ -18,19 +18,19 @@
package log
type Logger interface {
- WithField(key string, value interface{}) Logger
+ WithField(key string, value any) Logger
// Info logs to the INFO log.
- Info(args ...interface{})
+ Info(args ...any)
Review Comment:
PR title/description suggests a broad replacement of `interface{}` with
`any` across the codebase, but this PR (at least in the changes shown) only
updates the `log.Logger` interface. Consider updating the PR description/title
to reflect the narrower scope, or expanding the refactor to cover the remaining
`interface{}` usages if that was the intent.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
