Skip to content

Commit

Permalink
fix lint issues
Browse files Browse the repository at this point in the history
  • Loading branch information
danielstokes committed Jan 9, 2025
1 parent b80d56d commit fbaa05c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion internal/apm/helper.go
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,7 @@ func encodeAttributes(m map[string]string, fieldSeparator string, valueSeparator
str := ""
i := 0
keys := make([]string, len(m))
for key, _ := range m {
for key := range m {
keys[i] = key
i++
}
Expand Down
4 changes: 2 additions & 2 deletions internal/instrumentation/health_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -351,9 +351,9 @@ func TestHealthMonitor(t *testing.T) {
}
select {
case <-doneCh:
hm.Shutdown(context.Background())
_ = hm.Shutdown(context.Background())
case <-toCtx.Done():
hm.Stop(toCtx)
_ = hm.Stop(toCtx)
t.Fatal("toCtx timed out")
}
if diff := cmp.Diff(test.expectedInstrumentationStatus, instrumentationStatus, cmpopts.IgnoreFields(v1alpha2.InstrumentationStatus{}, "LastUpdated")); diff != "" {
Expand Down
2 changes: 1 addition & 1 deletion internal/instrumentation/util/worker/manyworkers_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ func TestManyWorkers(t *testing.T) {
go func() {
// add these in a go routine, because otherwise it will block until a worker can pick it up
for i := 0; i < 50; i++ {
w.Add(context.Background(), nil)
_ = w.Add(context.Background(), nil)
atomic.AddInt64(&jobsSent, 1)
}
}()
Expand Down

0 comments on commit fbaa05c

Please sign in to comment.