From a612cd2b7e53a6464fa9793080f8c5d9b702d4c8 Mon Sep 17 00:00:00 2001
From: Katrina Verey <katrina.verey@shopify.com>
Date: Mon, 28 Nov 2022 19:11:09 -0500
Subject: [PATCH] Fix pluginator incompatibility with nolint lines

---
 .../builtins/HelmChartInflationGenerator.go   |  5 +--
 .../internal/builtinplugin/builtinplugin.go   | 12 +++---
 .../internal/krmfunction/converter_test.go    |  6 +--
 .../krmfunction/funcwrappersrc/fakeplugin.go  |  3 +-
 .../AnnotationsTransformer.go                 |  3 +-
 .../configmapgenerator/ConfigMapGenerator.go  |  3 +-
 .../hashtransformer/HashTransformer.go        |  3 +-
 .../HelmChartInflationGenerator.go            | 40 +++++++++----------
 .../iampolicygenerator/IAMPolicyGenerator.go  |  3 +-
 .../ImageTagTransformer.go                    |  3 +-
 .../labeltransformer/LabelTransformer.go      |  3 +-
 .../LegacyOrderTransformer.go                 |  3 +-
 .../NamespaceTransformer.go                   |  3 +-
 .../PatchJson6902Transformer.go               |  3 +-
 .../PatchStrategicMergeTransformer.go         |  3 +-
 .../patchtransformer/PatchTransformer.go      |  3 +-
 .../prefixtransformer/PrefixTransformer.go    |  3 +-
 .../ReplacementTransformer.go                 |  3 +-
 .../ReplicaCountTransformer.go                |  3 +-
 .../secretgenerator/SecretGenerator.go        |  3 +-
 .../suffixtransformer/SuffixTransformer.go    |  3 +-
 .../ValueAddTransformer.go                    |  3 +-
 .../v1/calvinduplicator/CalvinDuplicator.go   |  4 +-
 .../v1/dateprefixer/DatePrefixer.go           |  8 ++--
 .../SecretsFromDatabase.go                    |  4 +-
 .../SomeServiceGenerator.go                   |  4 +-
 .../v1/stringprefixer/StringPrefixer.go       |  4 +-
 27 files changed, 54 insertions(+), 87 deletions(-)

diff --git a/api/internal/builtins/HelmChartInflationGenerator.go b/api/internal/builtins/HelmChartInflationGenerator.go
index b19d291bf..fd986da41 100644
--- a/api/internal/builtins/HelmChartInflationGenerator.go
+++ b/api/internal/builtins/HelmChartInflationGenerator.go
@@ -19,8 +19,7 @@ import (
 	"sigs.k8s.io/yaml"
 )
 
-// HelmChartInflationGeneratorPlugin is a plugin to generate resources
-// from a remote or local helm chart.
+// Generate resources from a remote or local helm chart.
 type HelmChartInflationGeneratorPlugin struct {
 	h *resmap.PluginHelpers
 	types.HelmGlobals
@@ -28,8 +27,6 @@ type HelmChartInflationGeneratorPlugin struct {
 	tmpDir string
 }
 
-var KustomizePlugin HelmChartInflationGeneratorPlugin
-
 const (
 	valuesMergeOptionMerge    = "merge"
 	valuesMergeOptionOverride = "override"
diff --git a/cmd/pluginator/internal/builtinplugin/builtinplugin.go b/cmd/pluginator/internal/builtinplugin/builtinplugin.go
index 809f74618..e6d66b8b0 100644
--- a/cmd/pluginator/internal/builtinplugin/builtinplugin.go
+++ b/cmd/pluginator/internal/builtinplugin/builtinplugin.go
@@ -72,10 +72,7 @@ func ConvertToBuiltInPlugin() (retErr error) {
 		if strings.HasPrefix(l, "//go:generate") {
 			continue
 		}
-		if strings.HasPrefix(l, "//noinspection") {
-			continue
-		}
-		if l == "var "+konfig.PluginSymbol+" plugin" {
+		if strings.HasPrefix(l, "var "+konfig.PluginSymbol+" plugin") {
 			continue
 		}
 		if strings.Contains(l, " Transform(") {
@@ -136,9 +133,12 @@ func newWriter(r string) (*writer, error) {
 }
 
 // Assume that this command is running with a $PWD of
-//   $HOME/kustomize/plugin/builtin/secretGenerator
+//
+//	$HOME/kustomize/plugin/builtin/secretGenerator
+//
 // (for example).  Then we want to write to
-//   $HOME/kustomize/api/builtins
+//
+//	$HOME/kustomize/api/builtins
 func makeOutputFileName(root string) string {
 	return filepath.Join(
 		"..", "..", "..", "api/internal", packageForGeneratedCode, root+".go")
diff --git a/cmd/pluginator/internal/krmfunction/converter_test.go b/cmd/pluginator/internal/krmfunction/converter_test.go
index f767eecea..4e48faa53 100644
--- a/cmd/pluginator/internal/krmfunction/converter_test.go
+++ b/cmd/pluginator/internal/krmfunction/converter_test.go
@@ -34,8 +34,7 @@ type plugin struct{
   FieldSpecs       []types.FieldSpec ` + "`json:\"fieldSpecs,omitempty\" yaml:\"fieldSpecs,omitempty\"`" + `
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
   _ *resmap.PluginHelpers, config []byte) (err error) {
@@ -171,8 +170,7 @@ type plugin struct {
 	types.ConfigMapArgs
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(h *resmap.PluginHelpers, config []byte) (err error) {
 	p.ConfigMapArgs = types.ConfigMapArgs{}
diff --git a/cmd/pluginator/internal/krmfunction/funcwrappersrc/fakeplugin.go b/cmd/pluginator/internal/krmfunction/funcwrappersrc/fakeplugin.go
index c8a71b649..895cfe59e 100644
--- a/cmd/pluginator/internal/krmfunction/funcwrappersrc/fakeplugin.go
+++ b/cmd/pluginator/internal/krmfunction/funcwrappersrc/fakeplugin.go
@@ -9,8 +9,7 @@ import (
 
 type plugin struct{}
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, _ []byte) (err error) {
diff --git a/plugin/builtin/annotationstransformer/AnnotationsTransformer.go b/plugin/builtin/annotationstransformer/AnnotationsTransformer.go
index 09b81bd8c..c6c8e74b5 100644
--- a/plugin/builtin/annotationstransformer/AnnotationsTransformer.go
+++ b/plugin/builtin/annotationstransformer/AnnotationsTransformer.go
@@ -17,8 +17,7 @@ type plugin struct {
 	FieldSpecs  []types.FieldSpec `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/configmapgenerator/ConfigMapGenerator.go b/plugin/builtin/configmapgenerator/ConfigMapGenerator.go
index 0ab9a7241..a2f835201 100644
--- a/plugin/builtin/configmapgenerator/ConfigMapGenerator.go
+++ b/plugin/builtin/configmapgenerator/ConfigMapGenerator.go
@@ -17,8 +17,7 @@ type plugin struct {
 	types.ConfigMapArgs
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(h *resmap.PluginHelpers, config []byte) (err error) {
 	p.ConfigMapArgs = types.ConfigMapArgs{}
diff --git a/plugin/builtin/hashtransformer/HashTransformer.go b/plugin/builtin/hashtransformer/HashTransformer.go
index 49861ce94..985c40f8c 100644
--- a/plugin/builtin/hashtransformer/HashTransformer.go
+++ b/plugin/builtin/hashtransformer/HashTransformer.go
@@ -15,8 +15,7 @@ type plugin struct {
 	hasher ifc.KustHasher
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	h *resmap.PluginHelpers, _ []byte) (err error) {
diff --git a/plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go b/plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go
index 5d41b3dcd..234856db9 100644
--- a/plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go
+++ b/plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go
@@ -23,17 +23,15 @@ import (
 	"sigs.k8s.io/yaml"
 )
 
-// HelmChartInflationGeneratorPlugin is a plugin to generate resources
-// from a remote or local helm chart.
-type HelmChartInflationGeneratorPlugin struct {
+// Generate resources from a remote or local helm chart.
+type plugin struct {
 	h *resmap.PluginHelpers
 	types.HelmGlobals
 	types.HelmChart
 	tmpDir string
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin HelmChartInflationGeneratorPlugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 const (
 	valuesMergeOptionMerge    = "merge"
@@ -49,7 +47,7 @@ var legalMergeOptions = []string{
 
 // Config uses the input plugin configurations `config` to setup the generator
 // options
-func (p *HelmChartInflationGeneratorPlugin) Config(
+func (p *plugin) Config(
 	h *resmap.PluginHelpers, config []byte) (err error) {
 	if h.GeneralConfig() == nil {
 		return fmt.Errorf("unable to access general config")
@@ -72,7 +70,7 @@ func (p *HelmChartInflationGeneratorPlugin) Config(
 // filesystem since we allow the user to use previously
 // downloaded charts.  This is safe since this plugin is
 // owned by kustomize.
-func (p *HelmChartInflationGeneratorPlugin) establishTmpDir() (err error) {
+func (p *plugin) establishTmpDir() (err error) {
 	if p.tmpDir != "" {
 		// already done.
 		return nil
@@ -81,7 +79,7 @@ func (p *HelmChartInflationGeneratorPlugin) establishTmpDir() (err error) {
 	return err
 }
 
-func (p *HelmChartInflationGeneratorPlugin) validateArgs() (err error) {
+func (p *plugin) validateArgs() (err error) {
 	if p.Name == "" {
 		return fmt.Errorf("chart name cannot be empty")
 	}
@@ -116,7 +114,7 @@ func (p *HelmChartInflationGeneratorPlugin) validateArgs() (err error) {
 	return nil
 }
 
-func (p *HelmChartInflationGeneratorPlugin) errIfIllegalValuesMerge() error {
+func (p *plugin) errIfIllegalValuesMerge() error {
 	if p.ValuesMerge == "" {
 		// Use the default.
 		p.ValuesMerge = valuesMergeOptionOverride
@@ -130,14 +128,14 @@ func (p *HelmChartInflationGeneratorPlugin) errIfIllegalValuesMerge() error {
 	return fmt.Errorf("valuesMerge must be one of %v", legalMergeOptions)
 }
 
-func (p *HelmChartInflationGeneratorPlugin) absChartHome() string {
+func (p *plugin) absChartHome() string {
 	if filepath.IsAbs(p.ChartHome) {
 		return p.ChartHome
 	}
 	return filepath.Join(p.h.Loader().Root(), p.ChartHome)
 }
 
-func (p *HelmChartInflationGeneratorPlugin) runHelmCommand(
+func (p *plugin) runHelmCommand(
 	args []string) ([]byte, error) {
 	stdout := new(bytes.Buffer)
 	stderr := new(bytes.Buffer)
@@ -163,7 +161,7 @@ func (p *HelmChartInflationGeneratorPlugin) runHelmCommand(
 }
 
 // createNewMergedValuesFile replaces/merges original values file with ValuesInline.
-func (p *HelmChartInflationGeneratorPlugin) createNewMergedValuesFile() (
+func (p *plugin) createNewMergedValuesFile() (
 	path string, err error) {
 	if p.ValuesMerge == valuesMergeOptionMerge ||
 		p.ValuesMerge == valuesMergeOptionOverride {
@@ -179,7 +177,7 @@ func (p *HelmChartInflationGeneratorPlugin) createNewMergedValuesFile() (
 	return p.writeValuesBytes(b)
 }
 
-func (p *HelmChartInflationGeneratorPlugin) replaceValuesInline() error {
+func (p *plugin) replaceValuesInline() error {
 	pValues, err := p.h.Loader().Load(p.ValuesFile)
 	if err != nil {
 		return err
@@ -200,7 +198,7 @@ func (p *HelmChartInflationGeneratorPlugin) replaceValuesInline() error {
 }
 
 // copyValuesFile to avoid branching.  TODO: get rid of this.
-func (p *HelmChartInflationGeneratorPlugin) copyValuesFile() (string, error) {
+func (p *plugin) copyValuesFile() (string, error) {
 	b, err := p.h.Loader().Load(p.ValuesFile)
 	if err != nil {
 		return "", err
@@ -209,7 +207,7 @@ func (p *HelmChartInflationGeneratorPlugin) copyValuesFile() (string, error) {
 }
 
 // Write a absolute path file in the tmp file system.
-func (p *HelmChartInflationGeneratorPlugin) writeValuesBytes(
+func (p *plugin) writeValuesBytes(
 	b []byte) (string, error) {
 	if err := p.establishTmpDir(); err != nil {
 		return "", fmt.Errorf("cannot create tmp dir to write helm values")
@@ -218,14 +216,14 @@ func (p *HelmChartInflationGeneratorPlugin) writeValuesBytes(
 	return path, errors.Wrap(os.WriteFile(path, b, 0644), "failed to write values file")
 }
 
-func (p *HelmChartInflationGeneratorPlugin) cleanup() {
+func (p *plugin) cleanup() {
 	if p.tmpDir != "" {
 		os.RemoveAll(p.tmpDir)
 	}
 }
 
 // Generate implements generator
-func (p *HelmChartInflationGeneratorPlugin) Generate() (rm resmap.ResMap, err error) {
+func (p *plugin) Generate() (rm resmap.ResMap, err error) {
 	defer p.cleanup()
 	if err = p.checkHelmVersion(); err != nil {
 		return nil, err
@@ -266,7 +264,7 @@ func (p *HelmChartInflationGeneratorPlugin) Generate() (rm resmap.ResMap, err er
 	return nil, err
 }
 
-func (p *HelmChartInflationGeneratorPlugin) templateCommand() []string {
+func (p *plugin) templateCommand() []string {
 	args := []string{"template"}
 	if p.ReleaseName != "" {
 		args = append(args, p.ReleaseName)
@@ -293,7 +291,7 @@ func (p *HelmChartInflationGeneratorPlugin) templateCommand() []string {
 	return args
 }
 
-func (p *HelmChartInflationGeneratorPlugin) pullCommand() []string {
+func (p *plugin) pullCommand() []string {
 	args := []string{
 		"pull",
 		"--untar",
@@ -308,7 +306,7 @@ func (p *HelmChartInflationGeneratorPlugin) pullCommand() []string {
 
 // chartExistsLocally will return true if the chart does exist in
 // local chart home.
-func (p *HelmChartInflationGeneratorPlugin) chartExistsLocally() (string, bool) {
+func (p *plugin) chartExistsLocally() (string, bool) {
 	path := filepath.Join(p.absChartHome(), p.Name)
 	s, err := os.Stat(path)
 	if err != nil {
@@ -318,7 +316,7 @@ func (p *HelmChartInflationGeneratorPlugin) chartExistsLocally() (string, bool)
 }
 
 // checkHelmVersion will return an error if the helm version is not V3
-func (p *HelmChartInflationGeneratorPlugin) checkHelmVersion() error {
+func (p *plugin) checkHelmVersion() error {
 	stdout, err := p.runHelmCommand([]string{"version", "-c", "--short"})
 	if err != nil {
 		return err
diff --git a/plugin/builtin/iampolicygenerator/IAMPolicyGenerator.go b/plugin/builtin/iampolicygenerator/IAMPolicyGenerator.go
index 12e19bef5..e05ba1f32 100644
--- a/plugin/builtin/iampolicygenerator/IAMPolicyGenerator.go
+++ b/plugin/builtin/iampolicygenerator/IAMPolicyGenerator.go
@@ -15,8 +15,7 @@ type plugin struct {
 	types.IAMPolicyGeneratorArgs
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(h *resmap.PluginHelpers, config []byte) (err error) {
 	p.IAMPolicyGeneratorArgs = types.IAMPolicyGeneratorArgs{}
diff --git a/plugin/builtin/imagetagtransformer/ImageTagTransformer.go b/plugin/builtin/imagetagtransformer/ImageTagTransformer.go
index c93e70cf1..8c8d9a7c6 100644
--- a/plugin/builtin/imagetagtransformer/ImageTagTransformer.go
+++ b/plugin/builtin/imagetagtransformer/ImageTagTransformer.go
@@ -18,8 +18,7 @@ type plugin struct {
 	FieldSpecs []types.FieldSpec `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/labeltransformer/LabelTransformer.go b/plugin/builtin/labeltransformer/LabelTransformer.go
index 513da6881..dd6e1600f 100644
--- a/plugin/builtin/labeltransformer/LabelTransformer.go
+++ b/plugin/builtin/labeltransformer/LabelTransformer.go
@@ -17,8 +17,7 @@ type plugin struct {
 	FieldSpecs []types.FieldSpec `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/legacyordertransformer/LegacyOrderTransformer.go b/plugin/builtin/legacyordertransformer/LegacyOrderTransformer.go
index ea7b8cbbc..13ae5f544 100644
--- a/plugin/builtin/legacyordertransformer/LegacyOrderTransformer.go
+++ b/plugin/builtin/legacyordertransformer/LegacyOrderTransformer.go
@@ -19,8 +19,7 @@ import (
 // (like ValidatingWebhookConfiguration) last.
 type plugin struct{}
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 // Nothing needed for configuration.
 func (p *plugin) Config(
diff --git a/plugin/builtin/namespacetransformer/NamespaceTransformer.go b/plugin/builtin/namespacetransformer/NamespaceTransformer.go
index e06e442bd..389579fe1 100644
--- a/plugin/builtin/namespacetransformer/NamespaceTransformer.go
+++ b/plugin/builtin/namespacetransformer/NamespaceTransformer.go
@@ -22,8 +22,7 @@ type plugin struct {
 	SetRoleBindingSubjects namespace.RoleBindingSubjectMode `json:"setRoleBindingSubjects" yaml:"setRoleBindingSubjects"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go b/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go
index 19f1c78d2..d3292da32 100644
--- a/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go
+++ b/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go
@@ -25,8 +25,7 @@ type plugin struct {
 	JsonOp       string          `json:"jsonOp,omitempty" yaml:"jsonOp,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	h *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go b/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go
index c566e2b35..abc440c0f 100644
--- a/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go
+++ b/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go
@@ -19,8 +19,7 @@ type plugin struct {
 	Patches       string                      `json:"patches,omitempty" yaml:"patches,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	h *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/patchtransformer/PatchTransformer.go b/plugin/builtin/patchtransformer/PatchTransformer.go
index 2d02cf28f..24b7b610b 100644
--- a/plugin/builtin/patchtransformer/PatchTransformer.go
+++ b/plugin/builtin/patchtransformer/PatchTransformer.go
@@ -26,8 +26,7 @@ type plugin struct {
 	Options      map[string]bool `json:"options,omitempty" yaml:"options,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	h *resmap.PluginHelpers, c []byte) error {
diff --git a/plugin/builtin/prefixtransformer/PrefixTransformer.go b/plugin/builtin/prefixtransformer/PrefixTransformer.go
index bb486b82e..c5899eb2d 100644
--- a/plugin/builtin/prefixtransformer/PrefixTransformer.go
+++ b/plugin/builtin/prefixtransformer/PrefixTransformer.go
@@ -20,8 +20,7 @@ type plugin struct {
 	FieldSpecs types.FsSlice `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 // TODO: Make this gvk skip list part of the config.
 var prefixFieldSpecsToSkip = types.FsSlice{
diff --git a/plugin/builtin/replacementtransformer/ReplacementTransformer.go b/plugin/builtin/replacementtransformer/ReplacementTransformer.go
index 1e4942949..90db9590f 100644
--- a/plugin/builtin/replacementtransformer/ReplacementTransformer.go
+++ b/plugin/builtin/replacementtransformer/ReplacementTransformer.go
@@ -20,8 +20,7 @@ type plugin struct {
 	Replacements    []types.Replacement      `json:"omitempty" yaml:"omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	h *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/replicacounttransformer/ReplicaCountTransformer.go b/plugin/builtin/replicacounttransformer/ReplicaCountTransformer.go
index e3bb8dcd9..a4d8eeb54 100644
--- a/plugin/builtin/replicacounttransformer/ReplicaCountTransformer.go
+++ b/plugin/builtin/replicacounttransformer/ReplicaCountTransformer.go
@@ -21,8 +21,7 @@ type plugin struct {
 	FieldSpecs []types.FieldSpec `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
diff --git a/plugin/builtin/secretgenerator/SecretGenerator.go b/plugin/builtin/secretgenerator/SecretGenerator.go
index a3c1955ad..6b31389b3 100644
--- a/plugin/builtin/secretgenerator/SecretGenerator.go
+++ b/plugin/builtin/secretgenerator/SecretGenerator.go
@@ -17,8 +17,7 @@ type plugin struct {
 	types.SecretArgs
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(h *resmap.PluginHelpers, config []byte) (err error) {
 	p.SecretArgs = types.SecretArgs{}
diff --git a/plugin/builtin/suffixtransformer/SuffixTransformer.go b/plugin/builtin/suffixtransformer/SuffixTransformer.go
index da01098c8..1b34b56df 100644
--- a/plugin/builtin/suffixtransformer/SuffixTransformer.go
+++ b/plugin/builtin/suffixtransformer/SuffixTransformer.go
@@ -20,8 +20,7 @@ type plugin struct {
 	FieldSpecs types.FsSlice `json:"fieldSpecs,omitempty" yaml:"fieldSpecs,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 // TODO: Make this gvk skip list part of the config.
 var suffixFieldSpecsToSkip = types.FsSlice{
diff --git a/plugin/builtin/valueaddtransformer/ValueAddTransformer.go b/plugin/builtin/valueaddtransformer/ValueAddTransformer.go
index ead170c06..b0c811b63 100644
--- a/plugin/builtin/valueaddtransformer/ValueAddTransformer.go
+++ b/plugin/builtin/valueaddtransformer/ValueAddTransformer.go
@@ -49,8 +49,7 @@ type Target struct {
 	FilePathPosition int `json:"filePathPosition,omitempty" yaml:"filePathPosition,omitempty"`
 }
 
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(h *resmap.PluginHelpers, c []byte) error {
 	err := yaml.Unmarshal(c, p)
diff --git a/plugin/someteam.example.com/v1/calvinduplicator/CalvinDuplicator.go b/plugin/someteam.example.com/v1/calvinduplicator/CalvinDuplicator.go
index 5a3b49e8e..276f46db1 100644
--- a/plugin/someteam.example.com/v1/calvinduplicator/CalvinDuplicator.go
+++ b/plugin/someteam.example.com/v1/calvinduplicator/CalvinDuplicator.go
@@ -22,9 +22,7 @@ type plugin struct {
 	Count int `json:"count,omitempty" yaml:"count,omitempty"`
 }
 
-//nolint: golint
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) Config(_ *resmap.PluginHelpers, c []byte) error {
 	return yaml.Unmarshal(c, p)
diff --git a/plugin/someteam.example.com/v1/dateprefixer/DatePrefixer.go b/plugin/someteam.example.com/v1/dateprefixer/DatePrefixer.go
index a416dbae5..5bf64c5c4 100644
--- a/plugin/someteam.example.com/v1/dateprefixer/DatePrefixer.go
+++ b/plugin/someteam.example.com/v1/dateprefixer/DatePrefixer.go
@@ -21,9 +21,7 @@ type plugin struct {
 	t resmap.Transformer
 }
 
-//nolint: golint
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) makePrefixPluginConfig() ([]byte, error) {
 	var s struct {
@@ -55,7 +53,9 @@ func (p *plugin) Config(h *resmap.PluginHelpers, _ []byte) error {
 }
 
 // Returns a constant, rather than
-//   time.Now().Format("2006-01-02")
+//
+//	time.Now().Format("2006-01-02")
+//
 // to make tests happy.
 // This is just an example.
 func getDate() string {
diff --git a/plugin/someteam.example.com/v1/secretsfromdatabase/SecretsFromDatabase.go b/plugin/someteam.example.com/v1/secretsfromdatabase/SecretsFromDatabase.go
index ae45bd57a..94ad032ae 100644
--- a/plugin/someteam.example.com/v1/secretsfromdatabase/SecretsFromDatabase.go
+++ b/plugin/someteam.example.com/v1/secretsfromdatabase/SecretsFromDatabase.go
@@ -19,9 +19,7 @@ type plugin struct {
 	Keys []string `json:"keys,omitempty" yaml:"keys,omitempty"`
 }
 
-//nolint: golint
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 var database = map[string]string{
 	"TREE":      "oak",
diff --git a/plugin/someteam.example.com/v1/someservicegenerator/SomeServiceGenerator.go b/plugin/someteam.example.com/v1/someservicegenerator/SomeServiceGenerator.go
index 62812edc0..2a69b2cca 100644
--- a/plugin/someteam.example.com/v1/someservicegenerator/SomeServiceGenerator.go
+++ b/plugin/someteam.example.com/v1/someservicegenerator/SomeServiceGenerator.go
@@ -19,9 +19,7 @@ type plugin struct {
 	Port             string `json:"port,omitempty" yaml:"port,omitempty"`
 }
 
-//nolint: golint
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 const tmpl = `
 apiVersion: v1
diff --git a/plugin/someteam.example.com/v1/stringprefixer/StringPrefixer.go b/plugin/someteam.example.com/v1/stringprefixer/StringPrefixer.go
index 073e7e99f..17355e798 100644
--- a/plugin/someteam.example.com/v1/stringprefixer/StringPrefixer.go
+++ b/plugin/someteam.example.com/v1/stringprefixer/StringPrefixer.go
@@ -22,9 +22,7 @@ type plugin struct {
 	t                resmap.Transformer
 }
 
-//nolint: golint
-//noinspection GoUnusedGlobalVariable
-var KustomizePlugin plugin
+var KustomizePlugin plugin //nolint:gochecknoglobals
 
 func (p *plugin) makePrefixPluginConfig(n string) ([]byte, error) {
 	var s struct {