Skip to content

Commit bfcd6ae

Browse files
committed
pkg/log: fix V return type to logr.Logger
Signed-off-by: Koichi Shiraishi <[email protected]>
1 parent d715d66 commit bfcd6ae

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

pkg/log/log_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,8 @@ func (f *fakeLogger) Info(msg string, vals ...interface{}) {
8585
})
8686
}
8787

88-
func (f *fakeLogger) Enabled() bool { return true }
89-
func (f *fakeLogger) V(lvl int) logr.InfoLogger { return f }
88+
func (f *fakeLogger) Enabled() bool { return true }
89+
func (f *fakeLogger) V(lvl int) logr.Logger { return f }
9090

9191
var _ = Describe("logging", func() {
9292

pkg/log/null.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,12 @@ type NullLogger struct{}
2929

3030
var _ logr.Logger = NullLogger{}
3131

32-
// Info implements logr.InfoLogger
32+
// Info implements logr.Logger
3333
func (NullLogger) Info(_ string, _ ...interface{}) {
3434
// Do nothing.
3535
}
3636

37-
// Enabled implements logr.InfoLogger
37+
// Enabled implements logr.Logger
3838
func (NullLogger) Enabled() bool {
3939
return false
4040
}
@@ -45,7 +45,7 @@ func (NullLogger) Error(_ error, _ string, _ ...interface{}) {
4545
}
4646

4747
// V implements logr.Logger
48-
func (log NullLogger) V(_ int) logr.InfoLogger {
48+
func (log NullLogger) V(_ int) logr.Logger {
4949
return log
5050
}
5151

pkg/log/zap/zap_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,8 @@ func (f *fakeLogger) Info(msg string, vals ...interface{}) {
112112
})
113113
}
114114

115-
func (f *fakeLogger) Enabled() bool { return true }
116-
func (f *fakeLogger) V(lvl int) logr.InfoLogger { return f }
115+
func (f *fakeLogger) Enabled() bool { return true }
116+
func (f *fakeLogger) V(lvl int) logr.Logger { return f }
117117

118118
var _ = Describe("Zap options setup", func() {
119119
var opts *Options

0 commit comments

Comments
 (0)