From e9be3251041a00a97a6594bbf93ea408ef14b016 Mon Sep 17 00:00:00 2001
From: Richard Wall <richard.wall@jetstack.io>
Date: Tue, 4 May 2021 14:30:33 +0100
Subject: [PATCH] Simplify the return statement

pkg/issuer/acme/dns/dns.go:164:2: should use 'return strategy == cmacme.FollowStrategy' instead of 'if strategy == cmacme.FollowStrategy { return true }; return false' (S1008)

Signed-off-by: Richard Wall <richard.wall@jetstack.io>
---
 pkg/issuer/acme/dns/dns.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/pkg/issuer/acme/dns/dns.go b/pkg/issuer/acme/dns/dns.go
index 76289f554..f40546c3e 100644
--- a/pkg/issuer/acme/dns/dns.go
+++ b/pkg/issuer/acme/dns/dns.go
@@ -161,10 +161,7 @@ func (s *Solver) CleanUp(ctx context.Context, issuer v1.GenericIssuer, ch *cmacm
 }
 
 func followCNAME(strategy cmacme.CNAMEStrategy) bool {
-	if strategy == cmacme.FollowStrategy {
-		return true
-	}
-	return false
+	return strategy == cmacme.FollowStrategy
 }
 
 func extractChallengeSolverConfig(ch *cmacme.Challenge) (*cmacme.ACMEChallengeSolverDNS01, error) {