fix already fixed checks that came in new

Author: purnesh42H

examples/features/advancedtls/client/main.go

@@ -16,6 +16,9 @@
  *
  */
 
+// Binary client is an example client demonstrating use of advancedtls,
+// to set up a secure gRPC client connection with various TLS authentication
+// methods.
 package main
 
 import (

examples/features/advancedtls/server/main.go

@@ -16,6 +16,8 @@
  *
  */
 
+// Binary server is an example client demonstrating how to set up a secure gRPC
+// server using advancedtls.
 package main
 
 import (

rpc_util.go

@@ -791,9 +791,9 @@ func checkRecvPayload(pf payloadFormat, recvCompress string, haveCompressor bool
 		if !haveCompressor {
 			if isServer {
 				return status.Newf(codes.Unimplemented, "grpc: Decompressor is not installed for grpc-encoding %q", recvCompress)
-			} else {
-				return status.Newf(codes.Internal, "grpc: Decompressor is not installed for grpc-encoding %q", recvCompress)
 			}
+			return status.Newf(codes.Internal, "grpc: Decompressor is not installed for grpc-encoding %q", recvCompress)
+
 		}
 	default:
 		return status.Newf(codes.Internal, "grpc: received unexpected payload format %d", pf)

scripts/revive.toml

@@ -3,21 +3,23 @@
 [rule.context-as-argument]
 [rule.context-keys-type]
 [rule.dot-imports]
+[rule.errorf]
 [rule.error-return]
 [rule.error-strings]
 [rule.error-naming]
 [rule.increment-decrement]
-[rule.var-naming]
-[rule.var-declaration]
+[rule.indent-error-flow]
+[rule.package-comments]
 [rule.range]
 [rule.receiver-naming]
+[rule.superfluous-else]
 [rule.time-naming]
+[rule.var-naming]
 [rule.unexported-return]
-[rule.indent-error-flow]
-[rule.errorf]
-[rule.superfluous-else]
 [rule.unused-parameter]
 [rule.unreachable-code]
+[rule.var-declaration]
+[rule.use-any]
 
 # Disabled rules
 # https://github.com/grpc/grpc-go/issues/7444 to track TODO(s)
@@ -26,8 +28,4 @@
 [rule.exported]
     Disabled = true # TODO: Enable after existing issues are fixed
 [rule.redefines-builtin-id]
-    Disabled = true # TODO: Enable after existing issues are fixed
-[rule.package-comments]
-    Disabled = true # TODO: Enable after existing issues are fixed
-[rule.use-any]
     Disabled = true # TODO: Enable after existing issues are fixed