Skip to content

Commit 78aa4d8

Browse files
committed
[chore](pkg): fix gofumpt issues part 1
fix gofumpt issues in pkg --- This commit was automatically created by review-helper. Changes are split into focused commits grouped by directory to simplify the review process. Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
1 parent a0187d0 commit 78aa4d8

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

pkg/collector/upgrade/versions.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,10 @@ import (
1111
)
1212

1313
// Deprecated use upgradeFuncV1beta1.
14-
type upgradeFunc func(u VersionUpgrade, otelcol *v1alpha1.OpenTelemetryCollector) (*v1alpha1.OpenTelemetryCollector, error)
15-
type upgradeFuncV1beta1 func(u VersionUpgrade, otelcol *v1beta1.OpenTelemetryCollector) (*v1beta1.OpenTelemetryCollector, error)
14+
type (
15+
upgradeFunc func(u VersionUpgrade, otelcol *v1alpha1.OpenTelemetryCollector) (*v1alpha1.OpenTelemetryCollector, error)
16+
upgradeFuncV1beta1 func(u VersionUpgrade, otelcol *v1beta1.OpenTelemetryCollector) (*v1beta1.OpenTelemetryCollector, error)
17+
)
1618

1719
type otelcolVersion struct {
1820
// deprecated use upgradeV1beta1.

pkg/sidecar/pod_test.go

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,8 @@ func TestExistsIn(t *testing.T) {
322322
pod corev1.Pod
323323
expected bool
324324
}{
325-
{"has-sidecar",
325+
{
326+
"has-sidecar",
326327
corev1.Pod{
327328
Spec: corev1.PodSpec{
328329
Containers: []corev1.Container{
@@ -331,9 +332,11 @@ func TestExistsIn(t *testing.T) {
331332
},
332333
},
333334
},
334-
true},
335+
true,
336+
},
335337

336-
{"does-have-native-sidecar",
338+
{
339+
"does-have-native-sidecar",
337340
corev1.Pod{
338341
Spec: corev1.PodSpec{
339342
Containers: []corev1.Container{
@@ -344,15 +347,18 @@ func TestExistsIn(t *testing.T) {
344347
},
345348
},
346349
},
347-
true},
350+
true,
351+
},
348352

349-
{"does-not-have-sidecar",
353+
{
354+
"does-not-have-sidecar",
350355
corev1.Pod{
351356
Spec: corev1.PodSpec{
352357
Containers: []corev1.Container{},
353358
},
354359
},
355-
false},
360+
false,
361+
},
356362
} {
357363
t.Run(tt.desc, func(t *testing.T) {
358364
useNativeSidecars := tt.desc == "does-have-native-sidecar"

0 commit comments

Comments
 (0)