enable enforce-switch-style linter

wandering-salesman · wandering-salesman · commit 3f32ac76e61f · 2025-08-31T23:22:27.000+05:30
Signed-off-by: Aditya &lt;jaiswal.aditya.deelip@gmail.com&gt;
diff --git a/.golangci.yml b/.golangci.yml
@@ -185,9 +185,6 @@ linters:
           # 2605 occurrences. Kind of useful in itself, but unacceptable amount of effort to fix
           disabled: true
         - name: enforce-switch-style
-          # Many occurrences.
-          disabled: true
-
     depguard:
       rules:
         no-patent:
diff --git a/cmd/nerdctl/compose/compose_pause_linux_test.go b/cmd/nerdctl/compose/compose_pause_linux_test.go
@@ -28,6 +28,7 @@ func TestComposePauseAndUnpause(t *testing.T) {
 	switch base.Info().CgroupDriver {
 	case "none", "":
 		t.Skip("requires cgroup (for pausing)")
+	default:
 	}
 
 	var dockerComposeYAML = fmt.Sprintf(`
diff --git a/cmd/nerdctl/container/container_create.go b/cmd/nerdctl/container/container_create.go
@@ -40,6 +40,7 @@ func CreateCommand() *cobra.Command {
 	case "freebsd":
 		longHelp += "\n"
 		longHelp += "WARNING: `nerdctl create` is experimental on FreeBSD and currently requires `--net=none` (https://github.com/containerd/nerdctl/blob/main/docs/freebsd.md)"
+	default:
 	}
 	var cmd = &cobra.Command{
 		Use:               "create [flags] IMAGE [COMMAND] [ARG...]",
diff --git a/cmd/nerdctl/container/container_run.go b/cmd/nerdctl/container/container_run.go
@@ -58,6 +58,7 @@ func RunCommand() *cobra.Command {
 	case "freebsd":
 		longHelp += "\n"
 		longHelp += "WARNING: `nerdctl run` is experimental on FreeBSD and currently requires `--net=none` (https://github.com/containerd/nerdctl/blob/main/docs/freebsd.md)"
+	default:
 	}
 	var cmd = &cobra.Command{
 		Use:               "run [flags] IMAGE [COMMAND] [ARG...]",
diff --git a/cmd/nerdctl/container/container_run_cgroup_linux_test.go b/cmd/nerdctl/container/container_run_cgroup_linux_test.go
@@ -53,6 +53,7 @@ func TestRunCgroupV2(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 
 	if !info.MemoryLimit {
@@ -152,6 +153,7 @@ func TestRunCgroupV1(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 	if !info.MemoryLimit {
 		t.Skip("test requires MemoryLimit")
@@ -195,6 +197,7 @@ func TestIssue3781(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 	containerName := testutil.Identifier(t)
 	base.Cmd("run", "-d", "--name", containerName, testutil.AlpineImage, "sleep", "infinity").AssertOK()
@@ -402,6 +405,7 @@ func TestRunCgroupConf(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 	if !info.MemoryLimit {
 		t.Skip("test requires MemoryLimit")
@@ -417,6 +421,7 @@ func TestRunCgroupParent(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 
 	containerName := testutil.Identifier(t)
@@ -476,6 +481,7 @@ func TestRunBlkioWeightCgroupV2(t *testing.T) {
 	switch info.CgroupDriver {
 	case "none", "":
 		t.Skip("test requires cgroup driver")
+	default:
 	}
 	containerName := testutil.Identifier(t)
 	defer base.Cmd("rm", "-f", containerName).AssertOK()
diff --git a/cmd/nerdctl/main_linux.go b/cmd/nerdctl/main_linux.go
@@ -51,6 +51,7 @@ func appNeedsRootlessParentMain(cmd *cobra.Command, args []string) bool {
 		switch commands[2] {
 		case "cp":
 			return false
+		default:
 		}
 	case "compose":
 		if len(commands) < 3 {
@@ -59,7 +60,9 @@ func appNeedsRootlessParentMain(cmd *cobra.Command, args []string) bool {
 		switch commands[2] {
 		case "cp":
 			return false
+		default:
 		}
+	default:
 	}
 	return true
 }
diff --git a/mod/tigron/.golangci.yml b/mod/tigron/.golangci.yml
@@ -97,7 +97,6 @@ linters:
             - "fmt.Fprintf"
         - name: redundant-test-main-exit
         - name: enforce-switch-style
-          disabled: true
         - name: var-naming
           disabled: true
     depguard:
diff --git a/mod/tigron/internal/com/command.go b/mod/tigron/internal/com/command.go
@@ -281,6 +281,7 @@ func (gc *Command) Wait() (*Result, error) {
 		return gc.result, gc.exec.err
 	case gc.result != nil:
 		return gc.result, ErrExecAlreadyFinished
+	default:
 	}
 
 	// Cancel the context in any case now
diff --git a/pkg/api/types/cri/metadata_types.go b/pkg/api/types/cri/metadata_types.go
@@ -70,6 +70,7 @@ func (c *ContainerMetadata) UnmarshalJSON(data []byte) error {
 	case metadataVersion:
 		*c = ContainerMetadata(versioned.Metadata)
 		return nil
+	default:
 	}
 	return fmt.Errorf("unsupported version: %q", versioned.Version)
 }
diff --git a/pkg/cmd/builder/build.go b/pkg/cmd/builder/build.go
@@ -390,6 +390,7 @@ func generateBuildctlArgs(ctx context.Context, client *containerd.Client, option
 			buildctlArgs = append(buildctlArgs, "--opt=image-resolve-mode=pull")
 		case false:
 			buildctlArgs = append(buildctlArgs, "--opt=image-resolve-mode=local")
+		default:
 		}
 	}
 
diff --git a/pkg/cmd/container/logs.go b/pkg/cmd/container/logs.go
@@ -48,6 +48,7 @@ func Logs(ctx context.Context, client *containerd.Client, container string, opti
 	switch options.GOptions.Namespace {
 	case "moby":
 		log.G(ctx).Warn("Currently, `nerdctl logs` only supports containers created with `nerdctl run -d` or CRI")
+	default:
 	}
 
 	stopChannel := make(chan os.Signal, 1)
diff --git a/pkg/cmd/container/run_mount.go b/pkg/cmd/container/run_mount.go
@@ -250,6 +250,7 @@ func generateMountOpts(ctx context.Context, client *containerd.Client, ensuredIm
 		switch imgVol {
 		case "/", "/dev", "/sys", "proc":
 			return nil, nil, nil, fmt.Errorf("invalid VOLUME: %q", imgVolRaw)
+		default:
 		}
 		if _, ok := mounted[imgVol]; ok {
 			continue
diff --git a/pkg/cmd/container/run_restart.go b/pkg/cmd/container/run_restart.go
@@ -32,6 +32,7 @@ func checkRestartCapabilities(ctx context.Context, client *containerd.Client, re
 	switch policySlice[0] {
 	case "", "no":
 		return true, nil
+	default:
 	}
 	res, err := client.IntrospectionService().Plugins(ctx, "id==restart")
 	if err != nil {
@@ -104,6 +105,7 @@ func UpdateContainerRestartPolicyLabel(ctx context.Context, client *containerd.C
 				desireStatus = containerd.Stopped
 			case containerd.Created:
 				desireStatus = containerd.Created
+			default:
 			}
 		}
 		updateOpts = append(updateOpts, restart.WithStatus(desireStatus))
diff --git a/pkg/cmd/container/top_unix.go b/pkg/cmd/container/top_unix.go
@@ -175,8 +175,9 @@ func fieldsASCII(s string) []string {
 		switch r {
 		case '\t', '\n', '\f', '\r', ' ':
 			return true
+		default:
+			return false
 		}
-		return false
 	}
 	return strings.FieldsFunc(s, fn)
 }
diff --git a/pkg/cmd/image/convert.go b/pkg/cmd/image/convert.go
@@ -179,6 +179,7 @@ func Convert(ctx context.Context, client *containerd.Client, srcRawRef, targetRa
 				Image: convertedRef,
 			}
 			return printConvertedImage(options.Stdout, options, res)
+		default:
 		}
 
 		if convertType != "overlaybd" {
diff --git a/pkg/cmd/network/inspect.go b/pkg/cmd/network/inspect.go
@@ -94,6 +94,7 @@ func Inspect(ctx context.Context, client *containerd.Client, options types.Netwo
 				return err
 			}
 			result = append(result, compat)
+		default:
 		}
 	}
 
diff --git a/pkg/cmd/network/list.go b/pkg/cmd/network/list.go
@@ -173,6 +173,8 @@ func getNetworkFilterFuncs(filters []string) ([]func(*map[string]string) bool, [
 					}
 					return true
 				})
+			default:
+				continue
 			}
 			continue
 		}
diff --git a/pkg/cmd/system/events.go b/pkg/cmd/system/events.go
@@ -90,6 +90,7 @@ func generateEventFilter(filter, filterValue string) (func(e *EventOut) bool, er
 
 			return strings.EqualFold(string(e.Status), filterValue)
 		}, nil
+	default:
 	}
 
 	return nil, fmt.Errorf("%s is an invalid or unsupported filter", filter)
diff --git a/pkg/cmd/system/info.go b/pkg/cmd/system/info.go
@@ -96,6 +96,7 @@ func Info(ctx context.Context, client *containerd.Client, options types.SystemIn
 		return prettyPrintInfoNative(options.Stdout, infoNative)
 	case "dockercompat":
 		return prettyPrintInfoDockerCompat(options.Stdout, options.Stderr, infoCompat, options.GOptions)
+	default:
 	}
 	return nil
 }
diff --git a/pkg/cmd/volume/list.go b/pkg/cmd/volume/list.go
@@ -242,6 +242,8 @@ func getVolumeFilterFuncs(filters []string) ([]func(*map[string]string) bool, []
 					}
 					return true
 				})
+			default:
+				continue
 			}
 			continue
 		}
diff --git a/pkg/composer/pipetagger/pipetagger.go b/pkg/composer/pipetagger/pipetagger.go
@@ -64,6 +64,7 @@ func ChooseColorAttrs(tag string) []color.Attribute {
 		attrs = append(attrs, color.BgHiWhite)
 	case color.FgHiWhite:
 		attrs = append(attrs, color.BgHiBlack)
+	default:
 	}
 
 	return attrs
diff --git a/pkg/dnsutil/hostsstore/hosts.go b/pkg/dnsutil/hostsstore/hosts.go
@@ -61,6 +61,7 @@ LINE:
 				skip = true
 			case MarkerEnd:
 				sawMarkerEnd = true
+			default:
 			}
 		}
 		if !skip {
diff --git a/pkg/infoutil/infoutil.go b/pkg/infoutil/infoutil.go
@@ -235,6 +235,7 @@ func parseRuncVersion(runcVersionStdout []byte) (*dockercompat.ComponentVersion,
 		switch strings.TrimSpace(detail[0]) {
 		case "commit":
 			details["GitCommit"] = strings.TrimSpace(detail[1])
+		default:
 		}
 	}
 
diff --git a/pkg/infoutil/infoutil_unix.go b/pkg/infoutil/infoutil_unix.go
@@ -84,6 +84,7 @@ func distroName(r io.Reader) string {
 			name = v
 		case "VERSION":
 			version = v
+		default:
 		}
 	}
 	if name != "" {
diff --git a/pkg/ipcutil/ipcutil.go b/pkg/ipcutil/ipcutil.go
@@ -207,6 +207,8 @@ func GenerateIPCOpts(ctx context.Context, ipc IPC, client *containerd.Client) ([
 		}
 
 		opts = append(opts, withBindMountHostOtherSourceIPC(*targetConIPC.HostShmPath))
+	default:
+		return nil, fmt.Errorf("unknown ipc mode: %s", ipc.Mode)
 	}
 
 	return opts, nil
@@ -242,6 +244,8 @@ func withBindMountHostIPC(_ context.Context, _ oci.Client, _ *containers.Contain
 				Source:      p,
 				Options:     []string{"rbind", "nosuid", "noexec", "nodev"},
 			}
+		default:
+			continue
 		}
 	}
 	return nil
diff --git a/pkg/manifestutil/manifestutils.go b/pkg/manifestutil/manifestutils.go
@@ -211,6 +211,8 @@ func CreateManifestEntry(parsedRef *referenceutil.ImageReference, desc ocispec.D
 		entry.SchemaV2Manifest = manifest
 	case "OCIManifest":
 		entry.OCIManifest = manifest
+	default:
+		return manifesttypes.DockerManifestEntry{}, fmt.Errorf("unsupported media type: %s", desc.MediaType)
 	}
 
 	// Special handling for OCI manifests to match Docker output
diff --git a/pkg/mountutil/mountutil.go b/pkg/mountutil/mountutil.go
@@ -136,6 +136,7 @@ func ProcessFlagV(s string, volStore volumestore.VolumeStore, createDir bool) (*
 			case "rbind", "bind":
 				fstype = "bind"
 				found = true
+			default:
 			}
 			if found {
 				break
diff --git a/pkg/mountutil/mountutil_linux.go b/pkg/mountutil/mountutil_linux.go
@@ -328,6 +328,7 @@ func ProcessFlagMount(s string, volStore volumestore.VolumeStore) (*Processed, e
 			case "bind-nonrecursive":
 				bindNonRecursive = true
 				continue
+			default:
 			}
 		}
 
@@ -386,7 +387,6 @@ func ProcessFlagMount(s string, volStore volumestore.VolumeStore) (*Processed, e
 
 	// compose new fileds and join into a string
 	// to call legacy ProcessFlagTmpfs or ProcessFlagV function
-	fields = []string{}
 	options := []string{}
 	if rwOption != "" {
 		if rwOption == "readonly" {
@@ -416,6 +416,8 @@ func ProcessFlagMount(s string, volStore volumestore.VolumeStore) (*Processed, e
 				options = append(options, "rbind")
 			}
 		}
+	default:
+		return nil, fmt.Errorf("invalid mount type '%s' must be a volume/bind/tmpfs", mountType)
 	}
 
 	if len(options) > 0 {
@@ -432,8 +434,9 @@ func ProcessFlagMount(s string, volStore volumestore.VolumeStore) (*Processed, e
 	case Volume, Bind:
 		// createDir=false for --mount option to disallow creating directories on host if not found
 		return ProcessFlagV(fieldsStr, volStore, false)
+	default:
+		return nil, fmt.Errorf("invalid mount type '%s' must be a volume/bind/tmpfs", mountType)
 	}
-	return nil, fmt.Errorf("invalid mount type '%s' must be a volume/bind/tmpfs", mountType)
 }
 
 // copy from https://github.com/moby/moby/blob/085c6a98d54720e70b28354ccec6da9b1b9e7fcf/volume/mounts/linux_parser.go#L375
diff --git a/pkg/platformutil/binfmt.go b/pkg/platformutil/binfmt.go
@@ -38,8 +38,9 @@ func qemuArchFromOCIArch(ociArch string) (string, error) {
 		return "mips64el", nil // NOT typo
 	case "loong64":
 		return "loongarch64", nil // NOT typo
+	default:
+		return "", fmt.Errorf("unknown OCI architecture string: %q", ociArch)
 	}
-	return "", fmt.Errorf("unknown OCI architecture string: %q", ociArch)
 }
 
 func canExecProbably(s string) (bool, error) {
diff --git a/pkg/reflectutil/reflectutil.go b/pkg/reflectutil/reflectutil.go
@@ -58,6 +58,7 @@ func isEmpty(v reflect.Value) bool {
 	switch v.Kind() {
 	case reflect.Map, reflect.Slice:
 		return v.Len() == 0
+	default:
+		return false
 	}
-	return false
 }
diff --git a/pkg/snapshotterutil/socisource.go b/pkg/snapshotterutil/socisource.go
@@ -104,6 +104,7 @@ func SociAppendDefaultLabelsHandlerWrapper(indexDigest string, wrapper func(imag
 						c.Annotations[targetImageLayersSizeLabel] = strings.TrimSuffix(layerSizes, ",")
 					}
 				}
+			default:
 			}
 			return children, nil
 		})
diff --git a/pkg/statsutil/stats_linux.go b/pkg/statsutil/stats_linux.go
@@ -175,6 +175,7 @@ func calculateCgroupBlockIO(metrics *v1.Metrics) (uint64, uint64) {
 			blkRead = blkRead + bioEntry.Value
 		case 'w', 'W':
 			blkWrite = blkWrite + bioEntry.Value
+		default:
 		}
 	}
 	return blkRead, blkWrite
diff --git a/pkg/testutil/nerdtest/requirements.go b/pkg/testutil/nerdtest/requirements.go
@@ -177,6 +177,7 @@ var CGroup = &test.Requirement{
 		case "none", "":
 			ret = false
 			mess = "cgroup is none"
+		default:
 		}
 		return ret, mess
 	},
@@ -212,6 +213,7 @@ var CGroupV2 = &test.Requirement{
 		case "none", "":
 			ret = false
 			mess = "cgroup is none"
+		default:
 		}
 		if dinf.CgroupVersion != "2" {
 			ret = false
diff --git a/pkg/testutil/testsyslog/testsyslog.go b/pkg/testutil/testsyslog/testsyslog.go
@@ -95,12 +95,15 @@ func TestableNetwork(network string) bool {
 			switch runtime.GOARCH {
 			case "arm", "arm64":
 				return false
+			default:
 			}
 		case "windows":
 			return false
+		default:
 		}
 	case "udp", "tcp", "tcp+tls":
 		return !rootlessutil.IsRootless()
+	default:
 	}
 	return true
 }
diff --git a/pkg/version/version.go b/pkg/version/version.go

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ func TestComposePauseAndUnpause(t *testing.T) {`
`28`	`28`	`switch base.Info().CgroupDriver {`
`29`	`29`	`case "none", "":`
`30`	`30`	`t.Skip("requires cgroup (for pausing)")`
	`31`	`+ default:`
`31`	`32`	`}`
`32`	`33`
`33`	`34`	var dockerComposeYAML = fmt.Sprintf(`
Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,7 @@ func CreateCommand() *cobra.Command {`
`40`	`40`	`case "freebsd":`
`41`	`41`	`longHelp += "\n"`
`42`	`42`	longHelp += "WARNING: `nerdctl create` is experimental on FreeBSD and currently requires `--net=none` (https://github.com/containerd/nerdctl/blob/main/docs/freebsd.md)"
	`43`	`+ default:`
`43`	`44`	`}`
`44`	`45`	`var cmd = &cobra.Command{`
`45`	`46`	`Use: "create [flags] IMAGE [COMMAND] [ARG...]",`
Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ func RunCommand() *cobra.Command {`
`58`	`58`	`case "freebsd":`
`59`	`59`	`longHelp += "\n"`
`60`	`60`	longHelp += "WARNING: `nerdctl run` is experimental on FreeBSD and currently requires `--net=none` (https://github.com/containerd/nerdctl/blob/main/docs/freebsd.md)"
	`61`	`+ default:`
`61`	`62`	`}`
`62`	`63`	`var cmd = &cobra.Command{`
`63`	`64`	`Use: "run [flags] IMAGE [COMMAND] [ARG...]",`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@ func appNeedsRootlessParentMain(cmd *cobra.Command, args []string) bool {`
`51`	`51`	`switch commands[2] {`
`52`	`52`	`case "cp":`
`53`	`53`	`return false`
	`54`	`+ default:`
`54`	`55`	`}`
`55`	`56`	`case "compose":`
`56`	`57`	`if len(commands) < 3 {`
`@@ -59,7 +60,9 @@ func appNeedsRootlessParentMain(cmd *cobra.Command, args []string) bool {`
`59`	`60`	`switch commands[2] {`
`60`	`61`	`case "cp":`
`61`	`62`	`return false`
	`63`	`+ default:`
`62`	`64`	`}`
	`65`	`+ default:`
`63`	`66`	`}`
`64`	`67`	`return true`
`65`	`68`	`}`
Original file line number	Diff line number	Diff line change
`@@ -281,6 +281,7 @@ func (gc Command) Wait() (Result, error) {`
`281`	`281`	`return gc.result, gc.exec.err`
`282`	`282`	`case gc.result != nil:`
`283`	`283`	`return gc.result, ErrExecAlreadyFinished`
	`284`	`+ default:`
`284`	`285`	`}`
`285`	`286`
`286`	`287`	`// Cancel the context in any case now`
Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,7 @@ func (c *ContainerMetadata) UnmarshalJSON(data []byte) error {`
`70`	`70`	`case metadataVersion:`
`71`	`71`	`*c = ContainerMetadata(versioned.Metadata)`
`72`	`72`	`return nil`
	`73`	`+ default:`
`73`	`74`	`}`
`74`	`75`	`return fmt.Errorf("unsupported version: %q", versioned.Version)`
`75`	`76`	`}`
Original file line number	Diff line number	Diff line change
`@@ -390,6 +390,7 @@ func generateBuildctlArgs(ctx context.Context, client *containerd.Client, option`
`390`	`390`	`buildctlArgs = append(buildctlArgs, "--opt=image-resolve-mode=pull")`
`391`	`391`	`case false:`
`392`	`392`	`buildctlArgs = append(buildctlArgs, "--opt=image-resolve-mode=local")`
	`393`	`+ default:`
`393`	`394`	`}`
`394`	`395`	`}`
`395`	`396`