chore: enable unnecessary-stmt rule from revive

Signed-off-by: Matthieu MOREL <[email protected]>

Author: mmorel-35

client/pkg/types/urls.go

@@ -47,7 +47,6 @@ func NewURLs(strs []string) (URLs, error) {
 				return nil, fmt.Errorf("URL must not contain a path: %s", in)
 			}
 		case "unix", "unixs":
-			break
 		default:
 			return nil, fmt.Errorf("URL scheme must be http, https, unix, or unixs: %s", in)
 		}

server/etcdmain/etcd.go

@@ -64,8 +64,7 @@ func startEtcdOrProxyV2(args []string) {
 	lg.Info("Running: ", zap.Strings("args", args))
 	if err != nil {
 		lg.Warn("failed to verify flags", zap.Error(err))
-		switch {
-		case errorspkg.Is(err, embed.ErrUnsetAdvertiseClientURLsFlag):
+		if errorspkg.Is(err, embed.ErrUnsetAdvertiseClientURLsFlag) {
 			lg.Warn("advertise client URLs are not set", zap.Error(err))
 		}
 		os.Exit(1)

server/storage/wal/repair.go

@@ -49,8 +49,7 @@ func Repair(lg *zap.Logger, dirpath string) bool {
 		switch {
 		case err == nil:
 			// update crc of the decoder when necessary
-			switch rec.Type {
-			case CrcType:
+			if rec.Type == CrcType {
 				crc := decoder.LastCRC()
 				// current crc of decoder must match the crc of the record.
 				// do no need to match 0 crc, since the decoder is a new one at this case.

tools/.golangci.yaml

@@ -44,58 +44,41 @@ linters-settings: # please keep this alphabetized
     # Align with https://github.com/alexkohler/nakedret/blob/v1.0.2/cmd/nakedret/main.go#L10
     max-func-lines: 5
   revive:
-    ignore-generated-header: false
-    severity: error
     confidence: 0.8
-    enable-all-rules: false
     rules:
       - name: blank-imports
-        severity: error
         disabled: false
       - name: context-as-argument
-        severity: error
+        disabled: false
+      - name: context-keys-type
         disabled: false
       - name: dot-imports
-        severity: error
+        disabled: false
+      - name: error-naming
         disabled: false
       - name: error-return
-        severity: error
         disabled: false
-      - name: error-naming
-        severity: error
+      - name: error-strings
+        disabled: false
+      - name: errorf
         disabled: false
       - name: if-return
-        severity: error
         disabled: false
       - name: increment-decrement
-        severity: error
         disabled: false
-      - name: var-declaration
-        severity: error
+      - name: indent-error-flow
         disabled: false
       - name: package-comments
-        severity: error
         disabled: false
       - name: range
-        severity: error
         disabled: false
       - name: receiver-naming
-        severity: error
         disabled: false
       - name: time-naming
-        severity: error
-        disabled: false
-      - name: indent-error-flow
-        severity: error
-        disabled: false
-      - name: errorf
-        severity: error
         disabled: false
-      - name: context-keys-type
-        severity: error
+      - name: unnecessary-stmt
         disabled: false
-      - name: error-strings
-        severity: error
+      - name: var-declaration
         disabled: false
       - name: var-naming
         disabled: false