fix lint findings from golangci-lint v2.12.2

Co-authored-by: mdelapenya <951580+mdelapenya@users.noreply.github.com>

Author: Copilot

config_test.go

@@ -13,9 +13,7 @@ import (
 func clearConfigEnv(t *testing.T) {
 	t.Helper()
 
-	var cfg config
-
-	typ := reflect.TypeOf(cfg)
+	typ := reflect.TypeFor[config]()
 	for i := range typ.NumField() {
 		field := typ.Field(i)
 		if name := field.Tag.Get("env"); name != "" {

consts.go

@@ -7,6 +7,9 @@ const (
 	// ryukLabel is the label used to identify reaper containers.
 	ryukLabel = labelBase + ".ryuk"
 
+	// labelValueTrue is the string value used for true labels.
+	labelValueTrue = "true"
+
 	// fieldError is the log field key for errors.
 	fieldError = "error"
 

reaper.go

@@ -453,7 +453,7 @@ func (r *reaper) affectedContainers(since time.Time, args filters.Args) ([]strin
 
 	containerIDs := make([]string, 0, len(containers))
 	for _, container := range containers {
-		if container.Labels[ryukLabel] == "true" {
+		if container.Labels[ryukLabel] == labelValueTrue {
 			// Ignore reaper containers.
 			r.logger.Debug("skipping reaper container", "id", container.ID)
 			continue

reaper_test.go

@@ -63,18 +63,18 @@ var (
 
 	// testLabels1 is a set of unique test labels.
 	testLabels1 = map[string]string{
-		labelBase:                "true",
+		labelBase:                labelValueTrue,
 		labelBase + ".sessionID": testID(),
 		labelBase + ".version":   "0.1.0",
-		labelBase + ".second":    "true",
+		labelBase + ".second":    labelValueTrue,
 	}
 
 	// testLabels2 is a second set of unique test labels.
 	testLabels2 = map[string]string{
-		labelBase:                "true",
+		labelBase:                labelValueTrue,
 		labelBase + ".sessionID": testID(),
 		labelBase + ".version":   "0.1.0",
-		labelBase + ".first":     "true",
+		labelBase + ".first":     labelValueTrue,
 	}
 
 	// mockContext is a matcher that matches any context.
@@ -733,6 +733,7 @@ func TestReapContainer(t *testing.T) {
 	cli := testClient(t)
 
 	ids := make([]string, 2)
+
 	for i, labels := range []map[string]string{testLabels1, testLabels2} {
 		config := &container.Config{
 			Image:  testImage,
@@ -788,6 +789,7 @@ func TestReapNetwork(t *testing.T) {
 	cli := testClient(t)
 
 	ids := make([]string, 2)
+
 	for i, labels := range []map[string]string{testLabels1, testLabels2} {
 		resp, err := cli.NetworkCreate(ctx, testID(), network.CreateOptions{
 			Labels: labels,
@@ -819,6 +821,7 @@ func TestReapVolume(t *testing.T) {
 	cli := testClient(t)
 
 	ids := make([]string, 2)
+
 	for i, labels := range []map[string]string{testLabels1, testLabels2} {
 		resp, err := cli.VolumeCreate(ctx, volume.CreateOptions{
 			Labels: labels,
@@ -850,6 +853,7 @@ func TestReapImage(t *testing.T) {
 	cli := testClient(t)
 
 	ids := make([]string, 2)
+
 	for i, labels := range []map[string]string{testLabels1, testLabels2} {
 		context, err := archive.Tar("testdata", archive.Uncompressed)
 		require.NoError(t, err)