From 5878f346bae8f575a59ef32ccf64d128da4d79af Mon Sep 17 00:00:00 2001
From: Wei Fu <fuweid89@gmail.com>
Date: Sat, 7 Oct 2023 22:27:28 +0800
Subject: [PATCH] *: migrate nakedret into golangci-lint

Signed-off-by: Wei Fu <fuweid89@gmail.com>
---
 scripts/test.sh      | 4 ----
 tools/.golangci.yaml | 4 ++++
 tools/mod/go.mod     | 1 -
 tools/mod/go.sum     | 2 --
 tools/mod/tools.go   | 1 -
 5 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/scripts/test.sh b/scripts/test.sh
index 8961ea837..b72883c55 100755
--- a/scripts/test.sh
+++ b/scripts/test.sh
@@ -438,10 +438,6 @@ function unconvert_pass {
   run_for_modules generic_checker run_go_tool "github.com/mdempsky/unconvert" unconvert -v
 }
 
-function nakedret_pass {
-  run_for_modules generic_checker run_go_tool "github.com/alexkohler/nakedret/cmd/nakedret"
-}
-
 function license_header_per_module {
   # bash 3.x compatible replacement of: mapfile -t gofiles < <(go_srcs_in_module)
   local gofiles=()
diff --git a/tools/.golangci.yaml b/tools/.golangci.yaml
index c917b536e..e3e734490 100644
--- a/tools/.golangci.yaml
+++ b/tools/.golangci.yaml
@@ -19,6 +19,7 @@ linters:
     # - varcheck
     - goimports
     - ineffassign
+    - nakedret
     - revive
     - staticcheck
     - stylecheck
@@ -27,6 +28,9 @@ linters:
 linters-settings: # please keep this alphabetized
   goimports:
     local-prefixes: go.etcd.io # Put imports beginning with prefix after 3rd-party packages.
+  nakedret:
+    # 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
diff --git a/tools/mod/go.mod b/tools/mod/go.mod
index c04d1f8bc..a999830fa 100644
--- a/tools/mod/go.mod
+++ b/tools/mod/go.mod
@@ -4,7 +4,6 @@ go 1.21
 
 require (
 	github.com/alexfalkowski/gocovmerge v1.2.1
-	github.com/alexkohler/nakedret v1.0.2
 	github.com/chzchzchz/goword v0.0.0-20170907005317-a9744cb52b03
 	github.com/cloudflare/cfssl v1.6.4
 	github.com/coreos/license-bill-of-materials v0.0.0-20190913234955-13baff47494e
diff --git a/tools/mod/go.sum b/tools/mod/go.sum
index 32e5959d4..d3372d00e 100644
--- a/tools/mod/go.sum
+++ b/tools/mod/go.sum
@@ -14,8 +14,6 @@ github.com/alecthomas/repr v0.2.0 h1:HAzS41CIzNW5syS8Mf9UwXhNH1J9aix/BvDRf1Ml2Yk
 github.com/alecthomas/repr v0.2.0/go.mod h1:Fr0507jx4eOXV7AlPV6AVZLYrLIuIeSOWtW57eE/O/4=
 github.com/alexfalkowski/gocovmerge v1.2.1 h1:A5j5eGZL1vwGmtMFu2pu9p2skraWTHTQTuf31Pc+Bvc=
 github.com/alexfalkowski/gocovmerge v1.2.1/go.mod h1:fKKMhPUyPBf/SF8xvMVkgpblgOT1zSXBK5+03gwgqJM=
-github.com/alexkohler/nakedret v1.0.2 h1:Svug5bxPnxbjw1jYlbHlnYAEORehXzsotyMUn2NQO2E=
-github.com/alexkohler/nakedret v1.0.2/go.mod h1:pRpUzThUf0nEk2mzur20zmf+6AVzw+3pDWQA0ehUZhI=
 github.com/bitfield/gotestdox v0.2.1 h1:Zj8IMLAO5/oiAKoMmtN96eyFiPZraJRTH2p0zDgtxc0=
 github.com/bitfield/gotestdox v0.2.1/go.mod h1:D+gwtS0urjBrzguAkTM2wodsTQYFHdpx8eqRJ3N+9pY=
 github.com/bmatcuk/doublestar/v4 v4.0.2/go.mod h1:xBQ8jztBU6kakFMg+8WGxn0c6z1fTSPVIjEY1Wr7jzc=
diff --git a/tools/mod/tools.go b/tools/mod/tools.go
index ebd9efcea..c34fe9059 100644
--- a/tools/mod/tools.go
+++ b/tools/mod/tools.go
@@ -22,7 +22,6 @@ package tools
 
 import (
 	_ "github.com/alexfalkowski/gocovmerge"
-	_ "github.com/alexkohler/nakedret"
 	_ "github.com/chzchzchz/goword"
 	_ "github.com/cloudflare/cfssl/cmd/cfssl"
 	_ "github.com/cloudflare/cfssl/cmd/cfssljson"