Merge pull request #29 from Boemeltrein/go-lint-fix2

fix(workflow): go-lint, error resolving

Author: alfi0812

cmd/flux.go

@@ -6,6 +6,7 @@ import (
     "github.com/spf13/cobra"
 )
 
+//nolint:unused
 var fluxLongHelp = strings.TrimSpace(`
 These are all commands that can be used to maintain FluxCD
 

cmd/helmrelease.go

@@ -6,6 +6,7 @@ import (
     "github.com/spf13/cobra"
 )
 
+//nolint:unused
 var helmreleaseHelp = strings.TrimSpace(`
 A toolkit to load helm-release files onto a cluster without flux.
 originally created to make it easier to install/upgrade/edit flux-based clusterresources without flux

cmd/talos.go

@@ -6,6 +6,7 @@ import (
     "github.com/spf13/cobra"
 )
 
+//nolint:unused
 var talosLongHelp = strings.TrimSpace(`
 These are all commands that can be used to maintain Talos OS
 

cmd/talos_kubeconfig.go

@@ -11,6 +11,7 @@ import (
 	"github.com/trueforge-org/forgetool/pkg/talassist"
 )
 
+//nolint:unused
 var advKubeconfigLongHelp = strings.TrimSpace(`
 
 `)

pkg/fluxhandler/helm.go

@@ -101,7 +101,7 @@ func HelmPull(repo string, name string, version string, dest string, silent bool
 		link = name
 		repoName := cleanRepoURL(repo)
 		updateHelmRepo(repoName, repo, silent)
-		repo = repoName
+		// repo = repoName  (Commented out by Boemeltrein, seems unused, changed for linting purposes)
 	} else {
 		link = repo + "/" + name
 		client.RepoURL = ""

pkg/fluxhandler/sshsecretgen.go

@@ -128,6 +128,7 @@ func CreateGitSecret(gitURL string) error {
 }
 
 // indentYaml indents each line of the YAML string with the specified indentation.
+//nolint:unused
 func indentYaml(yamlStr, indent string) string {
 	lines := strings.Split(yamlStr, "\n")
 	for i, line := range lines {
@@ -193,11 +194,13 @@ func generateKnownHostsEntry(url string) string {
 }
 
 // encodeToBase64 encodes data to a base64 string
+//nolint:unused
 func encodeToBase64(data []byte) string {
 	return string(data)
 }
 
 // decodeBase64 decodes a base64 string
+//nolint:unused
 func decodeBase64(data string) ([]byte, error) {
 	return []byte(data), nil
 }

pkg/gencmd/bootstrap.go

@@ -89,6 +89,12 @@ func RunBootstrap(args []string) {
 	helmRepoPath := filepath.Join("./repositories", "helm")
 	HelmRepos, err = fluxhandler.LoadAllHelmRepos(helmRepoPath)
 
+	// Added by Boemeltrein, for linting purposes
+	if err != nil {
+		log.Error().Err(err).Msg("Failed to load Helm repositories")
+		return
+	}
+
 	// Call ApprovePendingCertificates with clientset and stopCh
 	go kubectlcmds.ApprovePendingCertificates(clientset, stopCh)
 

pkg/gencmd/execcmd.go

@@ -15,6 +15,7 @@ func ExecCmd(cmd string) {
 	log.Trace().Msgf("command", argslice[:])
 
 	// log.Info().Msg("test", strings.Join(argslice, " "))
+	//nolint:ineffassign
 	out, err := helper.RunCommand(argslice, false)
 	if err != nil {
 		log.Info().Msgf("err:  %v", err)

pkg/initfiles/initfiles.go

@@ -488,6 +488,12 @@ func GetSecKey() (string, error) {
 func GenSopsSecret() error {
 	secretPath := filepath.Join(helper.ClusterPath, "kubernetes", "flux-system", "flux", "sopssecret.secret.yaml")
 	ageSecKey, err := GetSecKey()
+
+	// Added by Boemeltrein, for linting purposes
+	if err != nil {
+		return fmt.Errorf("failed to get age secret key: %w", err)
+	}
+
 	// Generate Kubernetes secret YAML content
 	secret := map[string]interface{}{
 		"apiVersion": "v1",

pkg/sops/wrapper.go

@@ -18,8 +18,10 @@ import (
 	"github.com/trueforge-org/forgetool/pkg/helper"
 )
 
+//nolint:unused
 var encrConfig *EncryptionConfig
 
+//nolint:unused
 const ageKeyFilePath = "./age.agekey"
 
 func EncryptWithAgeKey(body []byte, regex string, format string) ([]byte, error) {