Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit c79ddb3

Browse files
thockinthockin
committedApr 5, 2025··
Update to support golangci-lint v2
1 parent 20a64ba commit c79ddb3

File tree

10 files changed

+89
-84
lines changed

10 files changed

+89
-84
lines changed
 

‎.github/workflows/lint.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
go-version: '>=1.21.0'
1818
cache: false
1919
- name: Lint
20-
uses: golangci/golangci-lint-action@55c2c1448f86e01eaae002a5a3a9624417608d84 # v6.5.2
20+
uses: golangci/golangci-lint-action@1481404843c368bc19ca9406f87d6e0fc97bdcfd # v7.0.0
2121
with:
2222
# version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
2323
version: latest

‎.golangci.yaml

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,28 @@
1+
version: "2"
2+
13
run:
24
timeout: 1m
35
tests: true
46

57
linters:
6-
disable-all: true
7-
enable:
8+
default: none
9+
enable: # please keep this alphabetized
10+
- asasalint
811
- asciicheck
12+
- copyloopvar
13+
- dupl
914
- errcheck
1015
- forcetypeassert
16+
- goconst
1117
- gocritic
12-
- gofmt
13-
- goimports
14-
- gosimple
1518
- govet
1619
- ineffassign
1720
- misspell
21+
- musttag
1822
- revive
1923
- staticcheck
20-
- typecheck
2124
- unused
2225

2326
issues:
24-
exclude-use-default: false
2527
max-issues-per-linter: 0
2628
max-same-issues: 10

‎benchmark/benchmark_slog_test.go

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -32,42 +32,42 @@ import (
3232
//
3333

3434
func BenchmarkSlogSinkLogInfoOneArg(b *testing.B) {
35-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
35+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
3636
doInfoOneArg(b, log)
3737
}
3838

3939
func BenchmarkSlogSinkLogInfoSeveralArgs(b *testing.B) {
40-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
40+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
4141
doInfoSeveralArgs(b, log)
4242
}
4343

4444
func BenchmarkSlogSinkLogInfoWithValues(b *testing.B) {
45-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
45+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
4646
doInfoWithValues(b, log)
4747
}
4848

4949
func BenchmarkSlogSinkLogV0Info(b *testing.B) {
50-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
50+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
5151
doV0Info(b, log)
5252
}
5353

5454
func BenchmarkSlogSinkLogV9Info(b *testing.B) {
55-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
55+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
5656
doV9Info(b, log)
5757
}
5858

5959
func BenchmarkSlogSinkLogError(b *testing.B) {
60-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
60+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
6161
doError(b, log)
6262
}
6363

6464
func BenchmarkSlogSinkWithValues(b *testing.B) {
65-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
65+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
6666
doWithValues(b, log)
6767
}
6868

6969
func BenchmarkSlogSinkWithName(b *testing.B) {
70-
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard()))
70+
var log logr.Logger = logr.FromSlogHandler(logr.ToSlogHandler(logr.Discard())) //nolint:staticcheck
7171
doWithName(b, log)
7272
}
7373

@@ -82,61 +82,61 @@ func makeSlogJSONLogger() logr.Logger {
8282
}
8383

8484
func BenchmarkSlogJSONLogInfoOneArg(b *testing.B) {
85-
var log logr.Logger = makeSlogJSONLogger()
85+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
8686
doInfoOneArg(b, log)
8787
}
8888

8989
func BenchmarkSlogJSONLogInfoSeveralArgs(b *testing.B) {
90-
var log logr.Logger = makeSlogJSONLogger()
90+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
9191
doInfoSeveralArgs(b, log)
9292
}
9393

9494
func BenchmarkSlogJSONLogInfoWithValues(b *testing.B) {
95-
var log logr.Logger = makeSlogJSONLogger()
95+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
9696
doInfoWithValues(b, log)
9797
}
9898

9999
func BenchmarkSlogJSONLogV0Info(b *testing.B) {
100-
var log logr.Logger = makeSlogJSONLogger()
100+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
101101
doV0Info(b, log)
102102
}
103103

104104
func BenchmarkSlogJSONLogV9Info(b *testing.B) {
105-
var log logr.Logger = makeSlogJSONLogger()
105+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
106106
doV9Info(b, log)
107107
}
108108

109109
func BenchmarkSlogJSONLogError(b *testing.B) {
110-
var log logr.Logger = makeSlogJSONLogger()
110+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
111111
doError(b, log)
112112
}
113113

114114
func BenchmarkSlogJSONLogWithValues(b *testing.B) {
115-
var log logr.Logger = makeSlogJSONLogger()
115+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
116116
doWithValues(b, log)
117117
}
118118

119119
func BenchmarkSlogJSONWithName(b *testing.B) {
120-
var log logr.Logger = makeSlogJSONLogger()
120+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
121121
doWithName(b, log)
122122
}
123123

124124
func BenchmarkSlogJSONWithCallDepth(b *testing.B) {
125-
var log logr.Logger = makeSlogJSONLogger()
125+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
126126
doWithCallDepth(b, log)
127127
}
128128

129129
func BenchmarkSlogJSONLogInfoStringerValue(b *testing.B) {
130-
var log logr.Logger = makeSlogJSONLogger()
130+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
131131
doStringerValue(b, log)
132132
}
133133

134134
func BenchmarkSlogJSONLogInfoErrorValue(b *testing.B) {
135-
var log logr.Logger = makeSlogJSONLogger()
135+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
136136
doErrorValue(b, log)
137137
}
138138

139139
func BenchmarkSlogJSONLogInfoMarshalerValue(b *testing.B) {
140-
var log logr.Logger = makeSlogJSONLogger()
140+
var log logr.Logger = makeSlogJSONLogger() //nolint:staticcheck
141141
doMarshalerValue(b, log)
142142
}

‎benchmark/benchmark_test.go

Lines changed: 26 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -146,42 +146,42 @@ func doMarshalerValue(b *testing.B, log logr.Logger) {
146146
//
147147

148148
func BenchmarkDiscardLogInfoOneArg(b *testing.B) {
149-
var log logr.Logger = logr.Discard()
149+
var log logr.Logger = logr.Discard() //nolint:staticcheck
150150
doInfoOneArg(b, log)
151151
}
152152

153153
func BenchmarkDiscardLogInfoSeveralArgs(b *testing.B) {
154-
var log logr.Logger = logr.Discard()
154+
var log logr.Logger = logr.Discard() //nolint:staticcheck
155155
doInfoSeveralArgs(b, log)
156156
}
157157

158158
func BenchmarkDiscardLogInfoWithValues(b *testing.B) {
159-
var log logr.Logger = logr.Discard()
159+
var log logr.Logger = logr.Discard() //nolint:staticcheck
160160
doInfoWithValues(b, log)
161161
}
162162

163163
func BenchmarkDiscardLogV0Info(b *testing.B) {
164-
var log logr.Logger = logr.Discard()
164+
var log logr.Logger = logr.Discard() //nolint:staticcheck
165165
doV0Info(b, log)
166166
}
167167

168168
func BenchmarkDiscardLogV9Info(b *testing.B) {
169-
var log logr.Logger = logr.Discard()
169+
var log logr.Logger = logr.Discard() //nolint:staticcheck
170170
doV9Info(b, log)
171171
}
172172

173173
func BenchmarkDiscardLogError(b *testing.B) {
174-
var log logr.Logger = logr.Discard()
174+
var log logr.Logger = logr.Discard() //nolint:staticcheck
175175
doError(b, log)
176176
}
177177

178178
func BenchmarkDiscardWithValues(b *testing.B) {
179-
var log logr.Logger = logr.Discard()
179+
var log logr.Logger = logr.Discard() //nolint:staticcheck
180180
doWithValues(b, log)
181181
}
182182

183183
func BenchmarkDiscardWithName(b *testing.B) {
184-
var log logr.Logger = logr.Discard()
184+
var log logr.Logger = logr.Discard() //nolint:staticcheck
185185
doWithName(b, log)
186186
}
187187

@@ -193,91 +193,91 @@ func noopKV(_, _ string) {}
193193
func noopJSON(_ string) {}
194194

195195
func BenchmarkFuncrLogInfoOneArg(b *testing.B) {
196-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
196+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
197197
doInfoOneArg(b, log)
198198
}
199199

200200
func BenchmarkFuncrJSONLogInfoOneArg(b *testing.B) {
201-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
201+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
202202
doInfoOneArg(b, log)
203203
}
204204

205205
func BenchmarkFuncrLogInfoSeveralArgs(b *testing.B) {
206-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
206+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
207207
doInfoSeveralArgs(b, log)
208208
}
209209

210210
func BenchmarkFuncrJSONLogInfoSeveralArgs(b *testing.B) {
211-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
211+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
212212
doInfoSeveralArgs(b, log)
213213
}
214214

215215
func BenchmarkFuncrLogInfoWithValues(b *testing.B) {
216-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
216+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
217217
doInfoWithValues(b, log)
218218
}
219219

220220
func BenchmarkFuncrJSONLogInfoWithValues(b *testing.B) {
221-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
221+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
222222
doInfoWithValues(b, log)
223223
}
224224

225225
func BenchmarkFuncrLogV0Info(b *testing.B) {
226-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
226+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
227227
doV0Info(b, log)
228228
}
229229

230230
func BenchmarkFuncrJSONLogV0Info(b *testing.B) {
231-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
231+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
232232
doV0Info(b, log)
233233
}
234234

235235
func BenchmarkFuncrLogV9Info(b *testing.B) {
236-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
236+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
237237
doV9Info(b, log)
238238
}
239239

240240
func BenchmarkFuncrJSONLogV9Info(b *testing.B) {
241-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
241+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
242242
doV9Info(b, log)
243243
}
244244

245245
func BenchmarkFuncrLogError(b *testing.B) {
246-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
246+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
247247
doError(b, log)
248248
}
249249

250250
func BenchmarkFuncrJSONLogError(b *testing.B) {
251-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
251+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
252252
doError(b, log)
253253
}
254254

255255
func BenchmarkFuncrWithValues(b *testing.B) {
256-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
256+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
257257
doWithValues(b, log)
258258
}
259259

260260
func BenchmarkFuncrWithName(b *testing.B) {
261-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
261+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
262262
doWithName(b, log)
263263
}
264264

265265
func BenchmarkFuncrWithCallDepth(b *testing.B) {
266-
var log logr.Logger = funcr.New(noopKV, funcr.Options{})
266+
var log logr.Logger = funcr.New(noopKV, funcr.Options{}) //nolint:staticcheck
267267
doWithCallDepth(b, log)
268268
}
269269

270270
func BenchmarkFuncrJSONLogInfoStringerValue(b *testing.B) {
271-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
271+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
272272
doStringerValue(b, log)
273273
}
274274

275275
func BenchmarkFuncrJSONLogInfoErrorValue(b *testing.B) {
276-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
276+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
277277
doErrorValue(b, log)
278278
}
279279

280280
func BenchmarkFuncrJSONLogInfoMarshalerValue(b *testing.B) {
281-
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{})
281+
var log logr.Logger = funcr.NewJSON(noopJSON, funcr.Options{}) //nolint:staticcheck
282282
doMarshalerValue(b, log)
283283
}

‎examples/tab_logger.go

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@ See the License for the specific language governing permissions and
1414
limitations under the License.
1515
*/
1616

17+
// Package main implements a simple example of a logr.LogSink that logs to
18+
// stderr in a tabular format. It is not intended to be a production logger.
1719
package main
1820

1921
import (
@@ -36,23 +38,23 @@ var _ logr.LogSink = &tabLogSink{}
3638

3739
// Note that Init usually takes a pointer so it can modify the receiver to save
3840
// runtime info.
39-
func (*tabLogSink) Init(info logr.RuntimeInfo) {
41+
func (*tabLogSink) Init(_ logr.RuntimeInfo) {
4042
}
4143

42-
func (tabLogSink) Enabled(level int) bool {
44+
func (tabLogSink) Enabled(_ int) bool {
4345
return true
4446
}
4547

46-
func (l tabLogSink) Info(level int, msg string, kvs ...any) {
47-
fmt.Fprintf(l.writer, "%s\t%s\t", l.name, msg)
48+
func (l tabLogSink) Info(_ int, msg string, kvs ...any) {
49+
_, _ = fmt.Fprintf(l.writer, "%s\t%s\t", l.name, msg)
4850
for k, v := range l.keyValues {
49-
fmt.Fprintf(l.writer, "%s: %+v ", k, v)
51+
_, _ = fmt.Fprintf(l.writer, "%s: %+v ", k, v)
5052
}
5153
for i := 0; i < len(kvs); i += 2 {
52-
fmt.Fprintf(l.writer, "%s: %+v ", kvs[i], kvs[i+1])
54+
_, _ = fmt.Fprintf(l.writer, "%s: %+v ", kvs[i], kvs[i+1])
5355
}
54-
fmt.Fprintf(l.writer, "\n")
55-
l.writer.Flush()
56+
_, _ = fmt.Fprintf(l.writer, "\n")
57+
_ = l.writer.Flush()
5658
}
5759

5860
func (l tabLogSink) Error(err error, msg string, kvs ...any) {
@@ -74,7 +76,9 @@ func (l tabLogSink) WithValues(kvs ...any) logr.LogSink {
7476
newMap[k] = v
7577
}
7678
for i := 0; i < len(kvs); i += 2 {
77-
newMap[kvs[i].(string)] = kvs[i+1]
79+
k := kvs[i].(string) //nolint:forcetypeassert
80+
v := kvs[i+1]
81+
newMap[k] = v
7882
}
7983
return &tabLogSink{
8084
name: l.name,

‎examples/usage_example.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ var objectMap = map[string]Object{
5151
},
5252
}
5353

54-
// Object is an app contruct that might want to be logged.
54+
// Object is an app construct that might want to be logged.
5555
type Object struct {
5656
Name string
5757
Kind string
@@ -136,7 +136,7 @@ func (c *Controller) Run() {
136136
log.V(1).Info("reconciling object for key")
137137

138138
// Do some complicated updates updates
139-
obj.Details = obj.Details.(int) * 2
139+
obj.Details = obj.Details.(int) * 2 //nolint:forcetypeassert
140140

141141
// actually save the updates
142142
log.V(1).Info("updating object", "details", obj.Details)

‎funcr/example_formatter_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,17 +41,17 @@ type stdoutlogger struct {
4141
}
4242

4343
func (l stdoutlogger) WithName(name string) logr.LogSink {
44-
l.Formatter.AddName(name)
44+
l.AddName(name)
4545
return &l
4646
}
4747

4848
func (l stdoutlogger) WithValues(kvList ...any) logr.LogSink {
49-
l.Formatter.AddValues(kvList)
49+
l.AddValues(kvList)
5050
return &l
5151
}
5252

5353
func (l stdoutlogger) WithCallDepth(depth int) logr.LogSink {
54-
l.Formatter.AddCallDepth(depth)
54+
l.AddCallDepth(depth)
5555
return &l
5656
}
5757

‎funcr/example_test.go

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,11 @@ package funcr_test
1919
import (
2020
"fmt"
2121

22-
"github.com/go-logr/logr"
2322
"github.com/go-logr/logr/funcr"
2423
)
2524

2625
func ExampleNew() {
27-
var log logr.Logger = funcr.New(func(prefix, args string) {
26+
log := funcr.New(func(prefix, args string) {
2827
fmt.Println(prefix, args)
2928
}, funcr.Options{})
3029

@@ -35,7 +34,7 @@ func ExampleNew() {
3534
}
3635

3736
func ExampleNewJSON() {
38-
var log logr.Logger = funcr.NewJSON(func(obj string) {
37+
log := funcr.NewJSON(func(obj string) {
3938
fmt.Println(obj)
4039
}, funcr.Options{})
4140

@@ -46,7 +45,7 @@ func ExampleNewJSON() {
4645
}
4746

4847
func ExampleUnderlier() {
49-
var log logr.Logger = funcr.New(func(prefix, args string) {
48+
log := funcr.New(func(prefix, args string) {
5049
fmt.Println(prefix, args)
5150
}, funcr.Options{})
5251

@@ -58,7 +57,7 @@ func ExampleUnderlier() {
5857
}
5958

6059
func ExampleOptions() {
61-
var log logr.Logger = funcr.NewJSON(
60+
log := funcr.NewJSON(
6261
func(obj string) { fmt.Println(obj) },
6362
funcr.Options{
6463
LogCaller: funcr.All,
@@ -68,8 +67,8 @@ func ExampleOptions() {
6867
log.V(1).Info("V(1) message", "key", "value")
6968
log.V(2).Info("V(2) message", "key", "value")
7069
// Output:
71-
// {"logger":"","caller":{"file":"example_test.go","line":67},"level":0,"msg":"V(0) message","key":"value"}
72-
// {"logger":"","caller":{"file":"example_test.go","line":68},"level":1,"msg":"V(1) message","key":"value"}
70+
// {"logger":"","caller":{"file":"example_test.go","line":66},"level":0,"msg":"V(0) message","key":"value"}
71+
// {"logger":"","caller":{"file":"example_test.go","line":67},"level":1,"msg":"V(1) message","key":"value"}
7372
}
7473

7574
func ExampleOptions_renderHooks() {
@@ -87,7 +86,7 @@ func ExampleOptions_renderHooks() {
8786
return []any{"labels", funcr.PseudoStruct(kvList)}
8887
}
8988

90-
var log logr.Logger = funcr.NewJSON(
89+
log := funcr.NewJSON(
9190
func(obj string) { fmt.Println(obj) },
9291
funcr.Options{
9392
RenderBuiltinsHook: prefixSpecialKeys,
@@ -101,7 +100,7 @@ func ExampleOptions_renderHooks() {
101100
}
102101

103102
func ExamplePseudoStruct() {
104-
var log logr.Logger = funcr.NewJSON(
103+
log := funcr.NewJSON(
105104
func(obj string) { fmt.Println(obj) },
106105
funcr.Options{})
107106
kv := []any{
@@ -119,7 +118,7 @@ func ExampleOptions_maxLogDepth() {
119118
l := List{}
120119
l.Next = &l // recursive
121120

122-
var log logr.Logger = funcr.NewJSON(
121+
log := funcr.NewJSON(
123122
func(obj string) { fmt.Println(obj) },
124123
funcr.Options{MaxLogDepth: 4})
125124
log.Info("recursive", "list", l)

‎funcr/funcr.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ func newSink(fn func(prefix, args string), formatter Formatter) logr.LogSink {
7777
write: fn,
7878
}
7979
// For skipping fnlogger.Info and fnlogger.Error.
80-
l.Formatter.AddCallDepth(1)
80+
l.AddCallDepth(1) // via Formatter
8181
return l
8282
}
8383

@@ -164,17 +164,17 @@ type fnlogger struct {
164164
}
165165

166166
func (l fnlogger) WithName(name string) logr.LogSink {
167-
l.Formatter.AddName(name)
167+
l.AddName(name) // via Formatter
168168
return &l
169169
}
170170

171171
func (l fnlogger) WithValues(kvList ...any) logr.LogSink {
172-
l.Formatter.AddValues(kvList)
172+
l.AddValues(kvList) // via Formatter
173173
return &l
174174
}
175175

176176
func (l fnlogger) WithCallDepth(depth int) logr.LogSink {
177-
l.Formatter.AddCallDepth(depth)
177+
l.AddCallDepth(depth) // via Formatter
178178
return &l
179179
}
180180

‎testr/testr.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,12 +130,12 @@ type testloggerInterface struct {
130130
}
131131

132132
func (l testloggerInterface) WithName(name string) logr.LogSink {
133-
l.Formatter.AddName(name)
133+
l.AddName(name) // via Formatter
134134
return &l
135135
}
136136

137137
func (l testloggerInterface) WithValues(kvList ...any) logr.LogSink {
138-
l.Formatter.AddValues(kvList)
138+
l.AddValues(kvList) // via Formatter
139139
return &l
140140
}
141141

0 commit comments

Comments
 (0)
Please sign in to comment.